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

Editor Menubar Example: Visual design does not distinguish between menuitemcheckbox and menuitemradio elements #605

Closed
mcking65 opened this issue Feb 13, 2018 · 2 comments
Labels
enhancement Any addition or improvement that doesn't fix a code bug or prose inaccuracy Example Page Related to a page containing an example implementation of a pattern

Comments

@mcking65
Copy link
Contributor

In his review of pull request #593 for changes to the editor menubar example, @sh0ji commented:

There is no visual indicator that the submenu items are a checkbox or radio. You just have to test it or guess to know whether clicking one will de-select another.

@carmacleod commented:

I looked at the old Windows Notepad menu, and it has the same behavior, i.e. the user cannot tell, other than perhaps by context, that a menu item is a checkbox. For example, View -> Status Bar, or Format -> Word Wrap. I'm not sure what the best solution to this problem is; does Mac have anything nice? One possibility is to use a red X (or an empty box) when the checkbox is not selected, and a green checkmark (or green checkmark in a box) when it is selected. It would be nice to distinguish between radio and checkbox as well, so, for example, an empty circle (unselected) or blue dot within circle (selected) for radio.

Let's decide on an approach and fix this problem.

@mcking65 mcking65 added enhancement Any addition or improvement that doesn't fix a code bug or prose inaccuracy Example Page Related to a page containing an example implementation of a pattern labels Feb 13, 2018
@mcking65 mcking65 added this to the 1.1 APG Release 2 milestone Feb 13, 2018
carmacleod added a commit to carmacleod/aria-practices that referenced this issue Aug 22, 2018
carmacleod added a commit to carmacleod/aria-practices that referenced this issue Aug 22, 2018
@css-meeting-bot
Copy link
Member

The ARIA Authoring Practices (APG) Task Force just discussed Issue 605: distinguishing menuitemcheckbox from menuitemradio.

The full IRC log of that discussion <AnnAbbott> TOPIC: Issue 605: distinguishing menuitemcheckbox from menuitemradio
<MichielBijl> https://github.com//issues/605
<AnnAbbott> mck: requesting feedback
<mck> Link to proposal: https://rawgit.com/carmacleod/aria-practices/issue605/examples/menubar/menubar-2/menubar-2.html
<AnnAbbott> mck: should be able to distinguish between checkbox & radio in menu?
<AnnAbbott> mb: should be able to if grouped
<AnnAbbott> jn: no way to know a checkbox until checked
<AnnAbbott> jn: happy with example except off state doesn't display
<AnnAbbott> siri: off state should show up for kbd only users
<AnnAbbott> mb: does it display that way in native app?
<AnnAbbott> mck: when navigated, shouldn't only show on focus
<AnnAbbott> mck: only show on state?
<AnnAbbott> mck: take most accessible native approach unless compelling reason to do otherwise
<AnnAbbott> jg: could have different styling - just for selected state
<jamesn> GitHub: https://github.com//issues/605
<AnnAbbott> mb: agrees
<AnnAbbott> mck: comment in PR #853
<jemma> jemma: I agree with "too heavy lift example problem".
<AnnAbbott> jg: will comment in #853

carmacleod added a commit to carmacleod/aria-practices that referenced this issue Aug 27, 2018
…tween menuitemcheckbox and menuitemradio elements w3c#605"

This reverts commit e7a89e4
mcking65 pushed a commit that referenced this issue Dec 12, 2018
…radio and meuitemcheckbox (pull #853)

Resolves issue #605 by:
* Indicating menuitemcheckbox elements are checked with a  checkmark icon.
* Indicating menuitemradio elements are checked with a filled in circle icon.
@mcking65
Copy link
Contributor Author

@carmacleod, thank you very much for pull request #853.

@jongund, @sh0ji, and @jnurthen, thank you for your feedback on the pr.

I have squashed and merged the changes with commit 08e5fe0; this issue is now resolved, and the editor menubar continues to get better!

michael-n-cooper pushed a commit that referenced this issue Dec 12, 2018
Editor Menubar Example: Visual design to distinguish between menuitemradio and meuitemcheckbox (pull #853)

Resolves issue #605 by:
* Indicating menuitemcheckbox elements are checked with a  checkmark icon.
* Indicating menuitemradio elements are checked with a filled in circle icon.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Any addition or improvement that doesn't fix a code bug or prose inaccuracy Example Page Related to a page containing an example implementation of a pattern
Development

No branches or pull requests

2 participants