Skip to content

Add react-native-linear-gradient library#10902

Merged
lukewalczak merged 3 commits intogutenberg/after_1.22.0from
feat/linear-gradient
Feb 6, 2020
Merged

Add react-native-linear-gradient library#10902
lukewalczak merged 3 commits intogutenberg/after_1.22.0from
feat/linear-gradient

Conversation

@lukewalczak
Copy link
Copy Markdown
Contributor

@lukewalczak lukewalczak commented Dec 4, 2019

Fixes #

Ref to gb-mobile: wordpress-mobile/gutenberg-mobile#1616
Ref to gutenberg: WordPress/gutenberg#18823

This PR introduces 1st iteration of mobile Button block component.

To achieve gradient background color on mobile react-native-linear-gradient comes in handy.

The first iteration includes:

  • Rendering Button with:
    • gradient background color ✅
    • custom background color chosen from picker ✅
    • custom text color chosen from picker ✅
  • Options to change border radius, add link, open url in new tab ✅
  • Expanding and shrinking RichText
  • Notification for missing color settings functionality ✅

To test:

-> Gradient

  • open draft post on web
  • add Button
  • choose the gradient background color
  • save post
  • open draft post on mobile
  • check if a gradient is correct

Screenshot / gifs:

ANDROID

regular linear gradient
Screenshot 2019-12-02 at 10 35 50 Screenshot 2019-12-02 at 10 45 54

PR submission checklist:

  • I have considered adding unit tests where possible.

  • I have considered if this change warrants user-facing release notes and have added them to RELEASE-NOTES.txt if necessary.

@lukewalczak lukewalczak force-pushed the feat/linear-gradient branch 11 times, most recently from 48cda33 to 6d09331 Compare December 5, 2019 12:49
@peril-wordpress-mobile
Copy link
Copy Markdown

peril-wordpress-mobile bot commented Dec 5, 2019

You can test the changes on this Pull Request by downloading the APK here.

@lukewalczak lukewalczak marked this pull request as ready for review December 9, 2019 10:15
@lukewalczak lukewalczak force-pushed the feat/linear-gradient branch 2 times, most recently from 11a5698 to a75112c Compare January 17, 2020 14:47
@peril-wordpress-mobile
Copy link
Copy Markdown

Warnings
⚠️ PR is not assigned to a milestone.

Generated by 🚫 dangerJS

@pinarol
Copy link
Copy Markdown
Contributor

pinarol commented Feb 5, 2020

LGTM! Tested with the steps I mentioned in the gutenberg PR.

But, since the release branch is cut, we should merge into gutenberg/after_1.22.0 instead of develop.

(Another option is waiting until the release branch is merged into develop again which will happen next Monday)

@lukewalczak lukewalczak changed the base branch from develop to gutenberg/after_1.22.0 February 6, 2020 13:05
Copy link
Copy Markdown
Contributor

@pinarol pinarol left a comment

Choose a reason for hiding this comment

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

LGTM!

@lukewalczak lukewalczak merged commit eb31c5f into gutenberg/after_1.22.0 Feb 6, 2020
@lukewalczak lukewalczak deleted the feat/linear-gradient branch February 6, 2020 17:57
@hypest hypest mentioned this pull request Feb 11, 2020
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants