-
Notifications
You must be signed in to change notification settings - Fork 4.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[RNMobile] Create link settings component #23299
Conversation
Size Change: 0 B Total Size: 1.13 MB ℹ️ View Unchanged
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey Luke 👋
The code looks good! Nice to see some of the code moved into a new component to be reusable.
Regarding the sanity tests, I found one issue:
Button-6
- Buttons block - Settings: Open in new tab - steps
-
⚠️ Buttons block - Settings: Link rel - steps -> This one fails, I can't update the value. - Buttons block - Settings: Link URL - steps
- Buttons block - Settings: Remove link - steps
- Buttons block - Settings: Synchronize with button options - steps
Button-7
@geriux Thanks for CR! Will investigate it asap! |
@geriux Wrong value was passed to rel Would you mind running a check on that one more time? |
Sure! Can you please update the Gutenberg Mobile PR with develop? Since you merge with master I'd need the monorepo changes there. Thanks! |
Thanks for the changes!
LGTM! Tested it on the demo app and on WordPress iOS. |
That PR introduces a new component called
LinkSettings
to handle link configuration in blocks such asButtons
andSocial Link
and probably in a few more in the future.ref to gb mobile: wordpress-mobile/gutenberg-mobile#2406
Description
Refactored
Button
block to extract code related to link configuration and collect it in new component calledLinkSettings
.The most important prop in new component is
options
which is an object of options, eg:How has this been tested?
Go through test steps Button-6 and Button-7 from sanity-tests
Screenshots
Types of changes
Refactor - Button block
Feature - new component
LinkSettings
Checklist: