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

Add Ctrl-M to the tree drop down to open manage trees dialog #6269

Merged
merged 2 commits into from
Aug 13, 2023

Conversation

pHiney
Copy link
Contributor

@pHiney pHiney commented Jun 26, 2023

Fixes my annoyance at constantly clicking and scrolling to the bottom of a long list of trees, to find the "Manage Trees" entry and then possibly misclicking and having to do it all again.

Description of the problem being solved:

Technical solution to a management problem

Steps taken to verify a working solution:

  • Select the trees tab, select Ctrl-M. See that the "manage trees" dialog opens.
  • Select other tabs and select Ctrl-M. Confirm that the "manage trees" dialog DOES NOT open.

Question:

Do we want Ctrl-M or ctrl-m ??

Link to a build that showcases this PR:

N/A

Before screenshot:

image

After screenshot:

image

@Regisle
Copy link
Member

Regisle commented Jun 26, 2023

when adding hotkeys can you also add it to "help.txt" so we can keep better track to them? (and will need to update https://github.com/PathOfBuildingCommunity/PathOfBuilding/wiki/Keyboard-&-Mouse-Shortcuts when merged)

@pHiney
Copy link
Contributor Author

pHiney commented Jun 26, 2023

when adding hotkeys can you also add it to "help.txt" so we can keep better track to them?

didn't know about help.txt

@QuickStick123 QuickStick123 added the enhancement New feature, calculation, or mod label Aug 2, 2023
@LocalIdentity LocalIdentity merged commit 2431ccb into PathOfBuildingCommunity:dev Aug 13, 2023
@pHiney pHiney deleted the ctrl-m branch August 30, 2023 03:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature, calculation, or mod
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants