Skip to content

refactor: use styled-components in comment-list-item#723

Merged
machour merged 3 commits intogitpoint:masterfrom
chinesedfan:pr_557
Feb 13, 2018
Merged

refactor: use styled-components in comment-list-item#723
machour merged 3 commits intogitpoint:masterfrom
chinesedfan:pr_557

Conversation

@chinesedfan
Copy link
Copy Markdown
Member

@chinesedfan chinesedfan commented Feb 12, 2018

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

Screenshots

Before After
3 pic 2 pic

Description

Based on #557. Thank @MrLoh all the same.

Copy link
Copy Markdown
Member

@machour machour left a comment

Choose a reason for hiding this comment

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

Could you have a look at Travis?

},
});
const Container = styled.View`
padding: 10px 10px 0 0;
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.

This seems to cause a Travis build failure, could you have a look?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

@machour It is due to my stylelint regular expression. Fixed now.

`;

const ActionButtonIconContainer = styled.View`
padding: 5px 0 10px;
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.

And this too

@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage increased (+0.3%) to 44.478% when pulling 3382fdc on chinesedfan:pr_557 into a9639dc on gitpoint:master.

Copy link
Copy Markdown
Member

@machour machour left a comment

Choose a reason for hiding this comment

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

LGTM, tested on Android and iPhone simulators.

@machour machour merged commit 1947f1c into gitpoint:master Feb 13, 2018
@chinesedfan chinesedfan deleted the pr_557 branch July 12, 2019 02:36
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.

4 participants