-
Notifications
You must be signed in to change notification settings - Fork 4.3k
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
Paste: fix MS Word list paste #55127
Conversation
Size Change: +58 B (0%) Total Size: 1.65 MB
ℹ️ View Unchanged
|
Flaky tests detected in 7bb8955. 🔍 Workflow run URL: https://github.com/WordPress/gutenberg/actions/runs/6432419425
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I pasted the html provided on the issue on the trunk and verified the list got removed in this branch the issue is fixed 👍 The code also looks good.
41d8f6f
to
690cb02
Compare
* Paste: fix MS Word list paste * Match mso-list:Ignore * Fix inline paste
I just cherry-picked this PR to the 6.4-beta3-2 branch to get it included in the next release: d505480 |
* Site Editor Styles Screen: Fix dancing styles previews (#55183) * Pulling across changes from WordPress/wordpress-develop#5441 (#55181) Removed var * Add `aria-label` attribute to navigation block only when it is open (#54816) * Add `aria-label` only when is open * Remove unnecessary `label` property in edit * Escape translation * Move navigation context to `wp_json_encode` * Add `wp_json_encode` flags * Paste: fix MS Word list paste (#55127) * Paste: fix MS Word list paste * Match mso-list:Ignore * Fix inline paste * Fix scrollbars on pattern transforms (#55069) * Fix scrollbars on pattern transforms * Fix single pattern previews * Improve classname semantics * Remove modal title * Reset styles on window resize (#55077) Co-authored-by: Ricardo Artemio Morales <[email protected]> --------- Co-authored-by: Andrew Serong <[email protected]> Co-authored-by: Ramon <[email protected]> Co-authored-by: Mario Santos <[email protected]> Co-authored-by: Ella <[email protected]> Co-authored-by: Daniel Richards <[email protected]> Co-authored-by: Ricardo Artemio Morales <[email protected]>
What?
Fixes #46832.
Why?
The markup that MS Word gives us has changed. We're accidentally deleting the list text because previously the first span was the list number/bullet.
How?
Look for the
mso-list:Ignore
element.Testing Instructions
I added a new fixture.
Testing Instructions for Keyboard
Screenshots or screencast