-
Notifications
You must be signed in to change notification settings - Fork 4.2k
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
Code block: should use RichText (native) #56724
Conversation
Size Change: +489 B (0%) Total Size: 1.72 MB
ℹ️ View Unchanged
|
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.
LGTM! Thanks for updating this! I've added a few commits to fix some styling issues. With these changes we will be able to enable this block for production 🚀 (I'll do this in a separate PR).
Thanks @geriux! |
@ellatrix & @geriux can either of you explain why this change was needed. I need to do some further testing but this appears to be related to the Code Block being broken with I need to do some further investigation to see if there are actually changes to the RichText block that are causing the recent issue reported in the pending WP 6.5 release. Ref: #59548 |
To follow up, there is a string of changes around white space and the Ref: #58659 |
What?
Ideally both the mobile version of code uses the RichText just like the web version.
Related: #43204 (though we don't absolutely need this PR, I thought it's good to clean this up)
Why?
How?
Testing Instructions
Testing Instructions for Keyboard
Screenshots or screencast