-
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
Apply hover class in outline mode #27714
Conversation
Size Change: +26 B (0%) Total Size: 1.28 MB
ℹ️ View Unchanged
|
A little bit of off-topic: What's the difference between outline mode and navigate mode? 🤔 I'm trying to understand why outline mode is a thing. We already had navigate mode. Onto testing the changes. EDIT: It works fine! See review below |
isOutlineMode: getSettings().outlineMode, | ||
}; | ||
}, | ||
[ clientId ] |
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.
Is clientId
necessary as a dependency? It works fine without it.
I don't see it as three modes, I see it as two modes, select and edit.
This remains the same in site and post editors, the only difference is that we are experimenting with also showing the block boundary in edit mode for the site editor. If this is successful it could become an option to enable in either editor. |
Thanks for the review @david-szabo97, I pushed an update. |
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! 🚢 Left a minor suggestion.
Closes #27543
Restores the outline effect on hover when Site Editing:
To test:
is-hovered
class is not applied when hovering a blockis-hovered
class is applied when hovering a block