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

Sidebar header navigation improvements #1429

Closed
afercia opened this issue Jun 25, 2017 · 2 comments · Fixed by #1457
Closed

Sidebar header navigation improvements #1429

afercia opened this issue Jun 25, 2017 · 2 comments · Fixed by #1457
Assignees
Labels
[Type] Enhancement A suggestion for improvement.

Comments

@afercia
Copy link
Contributor

afercia commented Jun 25, 2017

Somehow related to #1356

It took me a while to understand that clicking the "Post" link in the Sidebar header:

screen shot 2017-06-25 at 14 55 08

switches back the sidebar to display the main settings instead of the specific setting of the block being edited:

screen shot 2017-06-25 at 14 54 16

It's a pretty neat feature, maybe a bit hidden and not so easily discoverable. Basically the Sidebar header behaves like sort of "breadcrumbs" navigation. Wondering if just renaming some controls would help making this feature more discoverable, for example changing the button label "Post Settings" and the header first level "Post" to just "Settings".

Worth also noting both terms are a bit misleading when editing a Page (and, in the future, CPTs):

screen shot 2017-06-25 at 14 56 00

screen shot 2017-06-25 at 14 56 37

@afercia afercia added the [Type] Enhancement A suggestion for improvement. label Jun 25, 2017
@jasmussen
Copy link
Contributor

Just to clarify, are you suggesting we rename the breadcrumb to "Post Settings" instead of just "Post" (and of course the CPTs)? If yes, I would agree that's a better experience 👍

@afercia
Copy link
Contributor Author

afercia commented Jun 26, 2017

Yep I think the "breadcrumbs" first level text (currently: "Post") should match the toggle button text and the default sidebar heading text. It could be "Post Settings" or maybe just "Settings": in the second case, there wouldn't be the need to distinguish between Post, Page, CPT...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Type] Enhancement A suggestion for improvement.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants