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

[iOS]Disable editing menu on title #747

Merged
merged 6 commits into from
Mar 15, 2019

Conversation

pinarol
Copy link
Contributor

@pinarol pinarol commented Mar 14, 2019

To Fix: #725

This PR removes formatting controls that appear on iOS side when we select text from post title.

gutenberg PR: WordPress/gutenberg#14435
WPiOS PR: wordpress-mobile/WordPress-iOS#11275

After the fix:

ipad-editing-menu-on-title

To Test

WPiOS

  • Checkout branch at [Gutenberg]Disable editing menu on post title WordPress-iOS#11275

  • Open the WPiOS app with an iPad

  • Select a text at title

  • Verify that formatting options aren't available in the menu above the keyboard

  • Verify that formatting options aren't available in the menu above the selected text

  • Select a text in a paragraph block

  • Verify that formatting options are available this time

WPAndroid << also important

  • Make sure gb editor in WPAndroid is running properly by doing some post updates

Copy link
Contributor

@etoledom etoledom left a comment

Choose a reason for hiding this comment

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

Working great! Thank you @pinarol

Tested on iPhone and iPad, WPiOS and example app.
Also WPAndroid looks good 👍

@pinarol pinarol merged commit da05401 into develop Mar 15, 2019
@pinarol pinarol deleted the issue/725-disable-editing-menu-on-title branch March 15, 2019 11:16
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.

[iOS] Formatting controls appear on post title text
2 participants