Skip to content

<Text> module add textDecoration style attributes - #1869

Closed
pcottle wants to merge 1 commit into
react:masterfrom
pcottle:squashRebaseKJRemote
Closed

<Text> module add textDecoration style attributes#1869
pcottle wants to merge 1 commit into
react:masterfrom
pcottle:squashRebaseKJRemote

Conversation

@pcottle

Copy link
Copy Markdown
Contributor

This is simply a rebased and squashed version of @KJlmfe's PR over at #845

It was actually already squashed into one commit, but for some reason that was hard to see from the original PR.

@facebook-github-botfacebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Jul 4, 2015
@pcottle

Copy link
Copy Markdown
ContributorAuthor
 ☂ -[UIExplorerTests testTextExampleSnapshot] (1208 ms) (0)

looks like the examples need to be re-recorded, justasec...

@pcottle
pcottleforce-pushed the squashRebaseKJRemote branch from 6507d52 to 6f240dbCompareJuly 4, 2015 18:10
@pcottle

Copy link
Copy Markdown
ContributorAuthor

Re-recorded tests, new image and results:

screen shot 2015-07-04 at 11 10 56 am

testtextexamplesnapshot_1 2x

@brentvatne

Copy link
Copy Markdown
Collaborator

Thanks @pcottle! This looks good to me

@brentvatne

Copy link
Copy Markdown
Collaborator

It would be great to get this in for 0.8.0-rc if it's ready, we are planning on shipping that sometime in the new couple of days if possible @nicklockwood (#1864)

@nicklockwood

Copy link
Copy Markdown
Contributor

Should be OK - I'll just need to check there are no issues with Android compatibility since it's a shared component.

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA SignedThis label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

@pcottle@brentvatne@nicklockwood@facebook-github-bot@KJlmfe