-
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
Add navigation editor block deselection when clicking canvas background #28876
Conversation
Size Change: +161 B (0%) Total Size: 1.37 MB
ℹ️ View Unchanged
|
316f1fe
to
a09c34a
Compare
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.
Very nice. Thank you!
(Sorry if I was the one that broke this previously.)
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 can't get used to the emptiness of the completely deselected state. Couldn't we instead have the nav block selected by default? This screen has no other function so it seems weird to have to select the nav block in order to interact with it.
The other thing is there's still a blue outline around the block after it's deselected:
@tellthemachines Yeah, I've suggested that as well in previous iterations. I'm open to trying that, but maybe in a separate PR. |
Description
Reimplements #28382, which had regressed (been accidentally removed).
How has this been tested?
e2e tests were already written and have been renabled
Manual testing:
Types of changes
Bug fix (non-breaking change which fixes an issue)
Checklist: