Skip to content

Conversation

@SergioEstevao
Copy link
Contributor

Description

This change makes sure that any extra strong element tags added on the post-title are removed correctly. This could have happened because of the presence of emoji characters on the title that made the RichText native element to add extra strong elements around the emojis.

How has this been tested?

This can be tested with this GB-mobile PR: wordpress-mobile/gutenberg-mobile#1930

Screenshots

Types of changes

Bug fix

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.
  • I've updated all React Native files affected by any refactorings/renamings in this PR.

@SergioEstevao SergioEstevao added [Type] Bug An existing feature does not function as intended Mobile App - i.e. Android or iOS Native mobile impl of the block editor. (Note: used in scripts, ping mobile folks to change) labels Feb 18, 2020
@SergioEstevao SergioEstevao added this to the Future milestone Feb 18, 2020
Copy link
Contributor

@mchowning mchowning left a comment

Choose a reason for hiding this comment

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

Tested and works great! 👍

@mchowning mchowning merged commit ef6c995 into master Feb 18, 2020
@mchowning mchowning deleted the rnmobile/fix_title_with_emoji_2 branch February 18, 2020 19:28
mchowning added a commit that referenced this pull request Feb 22, 2020
* Add view with accesibility label on top of floating toolbar (#20267)

* [FIX] inserter point in mobile app (#20195)

* fix inserter point in gutenberg-mobile

* Make sure that all strong tags are removed from title. (#20291)

Co-authored-by: Drapich Piotr <[email protected]>
Co-authored-by: Sérgio Estêvão <[email protected]>
@youknowriad youknowriad modified the milestones: Future, Gutenberg 7.6 Feb 26, 2020
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) [Type] Bug An existing feature does not function as intended

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants