Conversation
|
👋 Nice work, @SergioEstevao, I have updated the block blueprint - here is the latest:
Looks good to me other than those things! 👍 |
|
@iamthomasbishop, thanks for the feedback, one question though, if I apply your suggestion what will make it different from the pre block visually? After you write some text they will look exactly the same in the editor, no? |
The main diff is the background fill color. In your example the background looks white, and the block content area should have a dim color fill (one of the light grays in light mode, a dark gray in dark mode) to match the pre block style.
In terms of the contents of the block, static and selected states look the same, just like other text-based blocks. |
koke
left a comment
There was a problem hiding this comment.
Code-wise this looks good, except for one comment I left in the other PR. From what I understand this still needs some design updates, let me know when those are ready and I'll do some final testing
|
@koke design is updated to follow @iamthomasbishop recomendations, can you please give it another look? |
# Conflicts: # gutenberg
|
@koke I didn't change the font in Android, that still specifies monospace, that I'm assuming defaults to Roboto Mono. @iamthomasbishop do we want Menlo in Android to? |
|
@SergioEstevao I was assuming we'd use Menlo on iOS (considering it's either the default or at least an available font on iOS) and fall back to the default Roboto Mono on Android. Are we able to use Menlo on Android as well? |
|
@iamthomasbishop just confirmed with @maxme that for Android there is only one mono font by default and it's Roboto Mono. So we will keep that in Android and use Menlo in iOS. So all is ok with PR and I will merge it :). |
|
Sounds good! |


Fixes #368
This PR improves the style of the code block by:
Gutenberg PR: WordPress/gutenberg#18785
To test:
Update release notes:
RELEASE-NOTES.txt.