Skip to content
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

[Mobile] native mobile release v1.5.1 #15734

Merged
merged 4 commits into from May 20, 2019
Merged

[Mobile] native mobile release v1.5.1 #15734

merged 4 commits into from May 20, 2019

Conversation

hypest
Copy link
Contributor

@hypest hypest commented May 20, 2019

Description

PR to add a fix for an undo issue happening in the v1.5.0 release of mobile gutenberg. Related issue: wordpress-mobile/gutenberg-mobile#1001

This PR includes the 8160f88, 0e05d0c commits too, which have already been merged to master via #15703.

Note: this is a hotfix style PR and we should not use its merged state in the mobile side release, otherwise more GB commits will be brought in. So, we'll tag and use the hash of the 7631bb1 commit but not the one that will be created after merging this PR to master.

How has this been tested?

Using this gutenberg-mobile side PR: wordpress-mobile/gutenberg-mobile#1002

Types of changes

Inhibit setting the caret position if the position is larger than the text size.

Checklist:

  • My code is tested.
  • My code follows the WordPress code style.
  • My code follows the accessibility standards.
  • My code has proper inline documentation.
  • I've included developer documentation if appropriate.

etoledom and others added 3 commits May 17, 2019 10:03
* RichText: ownProps has precedence over block context.
In this way, instances of RichText outside blocks can pass and handle their own custom onFocus and isSelected

* RichText: Remove isSelected from block edit context.

* Fix lint issues
Removing it generates a regression where inserting a new list block, the block is not selected
@hypest hypest added the Mobile App - i.e. Android or iOS Native mobile impl of the block editor. (Note: used in scripts, ping mobile folks to change) label May 20, 2019
@hypest hypest requested a review from daniloercoli May 20, 2019 10:00
Copy link
Contributor

@daniloercoli daniloercoli left a comment

Choose a reason for hiding this comment

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

LGTM!

@hypest hypest merged commit d2a164b into master May 20, 2019
@hypest hypest deleted the rnmobile/release-v1.5.1 branch May 20, 2019 12:50
@youknowriad youknowriad added this to the 5.8 (Gutenberg) milestone May 24, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Mobile App - i.e. Android or iOS Native mobile impl of the block editor. (Note: used in scripts, ping mobile folks to change)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants