Skip to content

refactor(diff-blocks): switch to styled components#692

Merged
housseindjirdeh merged 1 commit intogitpoint:masterfrom
machour:diff-blocks-sc
Jan 16, 2018
Merged

refactor(diff-blocks): switch to styled components#692
housseindjirdeh merged 1 commit intogitpoint:masterfrom
machour:diff-blocks-sc

Conversation

@machour
Copy link
Copy Markdown
Member

@machour machour commented Jan 14, 2018

Question Response
Version? v1.4.1
Devices tested? iPhone 7
Bug fix? no
New feature? no
Includes tests? no
All Tests pass? yes
Related ticket? #532

Screenshots

Before After
screen shot 2018-01-14 at 12 09 13 pm screen shot 2018-01-14 at 12 05 18 pm
screen shot 2018-01-14 at 12 09 24 pm screen shot 2018-01-14 at 12 05 34 pm

Description

Migrated diff blocks to styled components.

(Reopened #533 as its repo was removed)

@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage increased (+0.3%) to 43.632% when pulling 90aeea1 on machour:diff-blocks-sc into 608818e on gitpoint:master.

Copy link
Copy Markdown
Member

@housseindjirdeh housseindjirdeh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Beautiful, thank you 🚀

let greyBlocks = null;
let addedBlocks = null;
let deletedBlocks = null;
let neutralBlocks = null;
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for this - definitely better variable names 👍

@housseindjirdeh housseindjirdeh merged commit c8e1a3d into gitpoint:master Jan 16, 2018
@machour machour deleted the diff-blocks-sc branch January 16, 2018 07:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants