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

Highlight text - Force format for some edge cases #4471

Merged
merged 2 commits into from
Jan 18, 2022

Conversation

geriux
Copy link
Contributor

@geriux geriux commented Jan 14, 2022

This PR fixes some edge cases when applying a highlighted text color format although there are still a few limitations on the native side (Aztec) but for most cases now it should work as expected.

To test check the Gutenberg PR description.

PR submission checklist:

  • I have considered adding unit tests where possible.
  • I have considered if this change warrants user-facing release notes more info and have added them to RELEASE-NOTES.txt if necessary.

@peril-wordpress-mobile
Copy link

peril-wordpress-mobile bot commented Jan 14, 2022

Wanna run full suite of Android and iOS UI tests? Click here and 'Approve' CI job!

@geriux geriux added the bugfix label Jan 14, 2022
@geriux geriux requested a review from antonis January 14, 2022 18:43
@geriux geriux marked this pull request as ready for review January 14, 2022 18:43
Copy link

@antonis antonis left a comment

Choose a reason for hiding this comment

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

Thank you for wrangling this @geriux 🙇
Approved via WordPress/gutenberg#37915 (review)

@geriux geriux added this to the 1.70.0 (19.1) milestone Jan 18, 2022
@geriux geriux force-pushed the fix/highlight-text-improvements branch from eece1b4 to 504d1f8 Compare January 18, 2022 10:52
@fluiddot
Copy link
Contributor

Hey @geriux, we will cut the 1.70.0 release on January 20th. I plan to circle back and bump this PR to the next milestone then, but please let me know if you’d rather us work to include this PR in 1.70.0, thanks 🙇.

@geriux geriux merged commit 8629e5c into develop Jan 18, 2022
@geriux geriux deleted the fix/highlight-text-improvements branch January 18, 2022 11:21
@fluiddot fluiddot mentioned this pull request Jan 20, 2022
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants