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

Inline boundaries don't work properly when using multiple inline formats #12973

Closed
youknowriad opened this issue Dec 18, 2018 · 1 comment · Fixed by #13697
Closed

Inline boundaries don't work properly when using multiple inline formats #12973

youknowriad opened this issue Dec 18, 2018 · 1 comment · Fixed by #13697
Labels
[Package] Rich text /packages/rich-text [Priority] High Used to indicate top priority items that need quick attention [Type] Bug An existing feature does not function as intended
Milestone

Comments

@youknowriad
Copy link
Contributor

Describe the bug

I have the following content in my post

<!-- wp:paragraph -->
<p><em>italic <code>code</code></em></p>
<!-- /wp:paragraph -->

Notice that it's impossible to move the caret after the code element, but before still inside the em to continue typing italic text.

Expected behavior
I expected the first right arrow key presse to go out of the code element but stay inside the em and the second one to go out of the em

@youknowriad youknowriad added [Type] Bug An existing feature does not function as intended [Feature] Writing Flow Block selection, navigation, splitting, merging, deletion... labels Dec 18, 2018
@youknowriad youknowriad added the [Priority] High Used to indicate top priority items that need quick attention label Jan 3, 2019
@ellatrix
Copy link
Member

ellatrix commented Jan 3, 2019

I wonder if this is an issue in TinyMCE inline mode as well. In any case, this will be resolved by #11322. Seems like I should prioritise that PR soon.

@ellatrix ellatrix added [Package] Rich text /packages/rich-text and removed [Feature] Writing Flow Block selection, navigation, splitting, merging, deletion... labels Jan 31, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Package] Rich text /packages/rich-text [Priority] High Used to indicate top priority items that need quick attention [Type] Bug An existing feature does not function as intended
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants