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

merge configuration related menu items under configuration submenu #3647

Closed
nvaccessAuto opened this issue Nov 14, 2013 · 4 comments
Closed

Comments

@nvaccessAuto
Copy link

Reported by surveyor on 2013-11-14 14:44
There are currently 4 configuration related menu items under NVDA menu, Configuration Profiles,
Revert to saved configuration,
Reset configuration to factory defaults,
Save configuration.
It maybe more convenient to move these items under Configuration submenu.

@nvaccessAuto
Copy link
Author

Comment 1 by jteh on 2013-11-14 23:10
I'm not sure i see any benefit to this; there aren't a huge number of items in the top level NVDA menu. Also, this might be confusing, since arguably preferences dialogs are related to configuration as well. That said, I'd be interested to hear others' thoughts.

@ehollig
Copy link
Collaborator

ehollig commented Aug 21, 2017

@jcsteh, do you still think something like this is counter-productive as stated in #5976 (comment)? Other comments in #5976 think this may be a useful feature.

@jcsteh
Copy link
Contributor

jcsteh commented Aug 22, 2017

  1. Change for change's sake is rarely a good idea.
  2. I don't think 11 items in the main menu is unreasonable, especially as these are pretty basic functions.
  3. IMO, adding an extra menu for these just makes them harder to find/get to.
  4. "Configuration" might get confused with "Preferences". If we were going to do this, I'd suggest "Configuration management".
  5. I'll defer to @Qchristensen, @michaelDCurran and @feerrenrut on this one; I'm not going to fight it if the consensus is to change it.

@michaelDCurran
Copy link
Member

I agree with Jamie's points. Closing as wontfix

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants