Skip to content
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

Chrome: Show block inspector when a block is selected #819

Merged
merged 2 commits into from
May 18, 2017

Conversation

youknowriad
Copy link
Contributor

This small PR shows the block inspector when a block is selected. If it's not the case, the post settings are shown in the sidebar.

@youknowriad youknowriad added the General Interface Parts of the UI which don't fall neatly under other labels. label May 17, 2017
@youknowriad youknowriad self-assigned this May 17, 2017
@jasmussen
Copy link
Contributor

jasmussen commented May 18, 2017

Works great! I love it. Exactly what I imagined. Would be good to have this in soon so we can test.

Not necessary for this branch, and it's blocked on #827: the underline in this mockup should actually be a clickable link, and clicking it should deselect any selected block:

screen shot 2017-05-18 at 10 56 02

The idea is that it's a breadcrumb, and clicking it takes you back to post settings.

👍 👍

@youknowriad
Copy link
Contributor Author

Thanks for the review @jasmussen I've updated the breadcrumb to unselect the post when we click "Post". I'll leave the "Post Settings" issue opened since this has other "focus" implications and would be good to tackle separately.

@mtias
Copy link
Member

mtias commented May 18, 2017

Looks good to me.

@youknowriad youknowriad merged commit 935b958 into master May 18, 2017
@youknowriad youknowriad deleted the update/block-inspector branch May 18, 2017 12:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
General Interface Parts of the UI which don't fall neatly under other labels.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants