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

Block "More" Menu: Final design proposal #7433

Closed
6 tasks done
jasmussen opened this issue Jun 21, 2018 · 8 comments
Closed
6 tasks done

Block "More" Menu: Final design proposal #7433

jasmussen opened this issue Jun 21, 2018 · 8 comments
Assignees
Labels
Needs Dev Ready for, and needs developer efforts [Priority] High Used to indicate top priority items that need quick attention [Type] Task Issues or PRs that have been broken down into an individual action to take
Milestone

Comments

@jasmussen
Copy link
Contributor

jasmussen commented Jun 21, 2018

Here's a mockup showing the latest iteration of the block's "More" menu:

screen shot 2018-06-21 at 14 56 38

Compare to master:

screen shot 2018-06-21 at 14 58 57

Note that although the mockup uses Gridicons, we'll keep using Dashicons in the menus. It's also okay to keep or adjust the labels in implementation, depending on prior history. Finally, the shortcut combinations are suggestions only, and we should adhere to standards where available, and consider other conflicts, when choosing these.

ToDo list:

@jasmussen
Copy link
Contributor Author

jasmussen commented Jun 21, 2018

It was suggested we add space between the trash button, if we add that to the menu, and the other menu items. Here's a pattern for separators in the popout menus:

screen shot 2018-06-21 at 15 51 39

Worth noting the above is an older mockup, and lacks the keyboard shortcuts that are actually shown in master:

screen shot 2018-06-21 at 15 54 31

@karmatosed karmatosed added Needs Design Feedback Needs general design feedback. and removed Design labels Jun 21, 2018
@karmatosed
Copy link
Member

This is a great iteration. I am not as keen on the inserting/adding but see the place for it so 👍 Let's try and get this in with the trash icon being there and a separator before it.

@mtias mtias added [Type] Task Issues or PRs that have been broken down into an individual action to take Design and removed Needs Design Feedback Needs general design feedback. labels Jun 22, 2018
@mtias
Copy link
Member

mtias commented Jun 22, 2018

Padding between item (focus ring) and the popover also stands out.

@karmatosed karmatosed added Needs Design Feedback Needs general design feedback. and removed Design labels Jun 25, 2018
@karmatosed karmatosed added Needs Dev Ready for, and needs developer efforts and removed Needs Design Feedback Needs general design feedback. labels Jul 15, 2018
@karmatosed karmatosed added this to the Merge Proposal: Editor milestone Jul 15, 2018
@mtias mtias added the [Priority] High Used to indicate top priority items that need quick attention label Aug 1, 2018
@talldan talldan self-assigned this Aug 2, 2018
@jasmussen
Copy link
Contributor Author

CC: @talldan regarding keyboard shortcuts, I just updated this issue. I will make a separate PR for padding, and I'm going to create some Dashicons as well.

This was referenced Aug 9, 2018
jasmussen pushed a commit that referenced this issue Aug 9, 2018
The padding was actually 11px, now that's compensated for.

Also sort the Edit as HTML button at the end of the first list. See #7433
@jasmussen
Copy link
Contributor Author

I've added pull requests to check a number of these boxes, and there's now also a PR to add the new icons in WordPress/dashicons#311.

@talldan
Copy link
Contributor

talldan commented Aug 9, 2018

Thanks @jasmussen - really appreciate you looking at those separately.

@noisysocks
Copy link
Member

What's left to do here?

@jasmussen
Copy link
Contributor Author

Yay, it's done! Closing as such.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Needs Dev Ready for, and needs developer efforts [Priority] High Used to indicate top priority items that need quick attention [Type] Task Issues or PRs that have been broken down into an individual action to take
Projects
None yet
Development

No branches or pull requests

5 participants