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.

Sign up for GitHub

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

Jump to bottom

Change main menu accelerators #197

Closed
ProgerXP opened this issue Nov 18, 2018 · 7 comments
Closed

Change main menu accelerators #197

ProgerXP opened this issue Nov 18, 2018 · 7 comments
Assignees
@ProgerXP
Labels
upcoming release

Comments

@ProgerXP
Copy link
Owner

ProgerXP commented Nov 18, 2018 •
edited
Loading

Remove accelerators from menu items (of the top submenu only) which have another hotkey. This is needed because there are many menu items and we waste available letters on commands that can be called with a global hotkey.

This means removing accelerators from:

  • File > New, Open, Revert, 3 Save [...], Rename, Print, Recent
    • keep E&xit for traditional compatibility
    • also change: Save On &Lose Focus
    • submenus (Launch, etc.) here and in other menus are untouched
  • Edit - all except Clear Clipboard and 5 commands with submenus
  • View - untouched, it's rarely used, small enough and has no accelerator conflicts
  • Settings - Tab Settings, Auto Close, Always On Top, Transparent Mode, File Change, 3 last settings-related commands
    • change: Evaluate &Math Expressions, C&trl+Wheel, S&how Toolbar
  • ? - untouched
cshnik added a commit that referenced this issue Nov 27, 2018
@cshnik
Change main menu accelerators #197
cbf3d87
@cshnik
Copy link
Collaborator

cshnik commented Nov 27, 2018

Done.

@cshnik cshnik assigned ProgerXP and unassigned cshnik Nov 27, 2018
@ProgerXP
Copy link
Owner Author

What about File > Revert?

@ProgerXP ProgerXP assigned cshnik and unassigned ProgerXP Nov 28, 2018
cshnik added a commit that referenced this issue Nov 30, 2018
@cshnik
Change main menu accelerators #197 (fix for File>Revert)
581abb7
@cshnik
Copy link
Collaborator

cshnik commented Nov 30, 2018

Fixed.

@cshnik cshnik assigned ProgerXP and unassigned cshnik Nov 30, 2018
@ProgerXP
Copy link
Owner Author

ProgerXP commented Dec 1, 2018

My bad, I have missed several more clashes:

  • Rename Save On Lose Focus to ... Fo&cus
  • Rename Edit > &Lines
  • Remove & entirely from 6 items in: Edit > Special > String To Hex ... untill and including QP Decode
  • Add to and rename ? > &3rd-Party Code

@ProgerXP ProgerXP assigned cshnik and unassigned ProgerXP Dec 1, 2018
cshnik added a commit that referenced this issue Dec 4, 2018
@cshnik
Change main menu accelerators #197
8a9bc28
@cshnik
Copy link
Collaborator

cshnik commented Dec 4, 2018

Fixed.

@cshnik cshnik assigned ProgerXP and unassigned cshnik Dec 4, 2018
@ProgerXP ProgerXP closed this as completed Dec 4, 2018
@ProgerXP
Copy link
Owner Author

ProgerXP commented Jul 5, 2019

  • Remove & from two Edit > Encode > URL ...
  • Rename ? > &3rd-Party Code (3rd plus dash: "third-party")

@ProgerXP ProgerXP reopened this Jul 5, 2019
@ProgerXP ProgerXP assigned cshnik and unassigned ProgerXP Jul 5, 2019
cshnik added a commit that referenced this issue Jul 5, 2019
@cshnik
Change main menu accelerators #197
53f6a52
@cshnik
Copy link
Collaborator

cshnik commented Jul 5, 2019

Fixed.

@cshnik cshnik assigned ProgerXP and unassigned cshnik Jul 5, 2019
@ProgerXP ProgerXP closed this as completed Jul 5, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ProgerXP ProgerXP

Labels
upcoming release
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@ProgerXP @cshnik