-
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
[RNMobile] Update native BlockOutline component styles #51222
Conversation
Size Change: +53 B (0%) Total Size: 1.42 MB
ℹ️ View Unchanged
|
Hey @derekblank 👋 while doing a quick test of some E2E tests I saw an issue with the Columns block UI that we might need to find a way to solve somehow: Screen.Recording.2023-06-21.at.09.22.08.movIt looks like the dashed outline is hidden when the parent block is selected, I think we might need to show the dashed outline for this case so it doesn't look empty (at least when there aren't any blocks in the content) what do you think? We might need design feedback here. |
# Conflicts: # packages/react-native-editor/CHANGELOG.md
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! Nice work! I've added a few changes related to the post title vertical padding and the block outline for the Colums block.
What?
Updates the native BlockOutline component and supporting styles.
Related:
Why?
As part of ongoing editor UX improvements, the solid border around outer blocks and dashed border around inner blocks are being removed.
How?
Update styles and references to
BlockOutline
component so that the main blue border around blocks is removed.Testing Instructions
Screenshots
NoOutlineTestBuild.1.mov