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

[WebUI 2.2.5] Hotkeys Update #1965

Merged
merged 4 commits into from
Dec 13, 2022
Merged

[WebUI 2.2.5] Hotkeys Update #1965

merged 4 commits into from
Dec 13, 2022

Conversation

blessedcoolant
Copy link
Collaborator

Changes in this PR

  • Hotkeys for Upscale and Restore have been changed from U and R to Shift+U and Shift+R respectively. The changes for users accidentally pressing R and U to trigger these functions was annoying. Especially considering R is also a shared hotkey with Reset View that easily becomes muscle memory -- especially in the viewer.
  • Updated Hotkeys modal to reflect these changes and improved minor descriptions related to the staging area hotkeys.
  • Updated the WEBUIHOTKEYS doc. It was outdated and not updated to reflect the changes made in the 2.2 Update.

@blessedcoolant
Change Restore and Upscale Hotkeys
eb88669 
Changed the hotkeys of Restore and Upscale from R and U to Shift R and Shift U. Users could accidentally press R and U to trigger these functions which can be annoying. Especially considering R is also a hotkey for Reset View in other tabs and it can become muscle memory.
@blessedcoolant
Update Hotkeys Modal
02b1447 
Update hotkeys modal to reflect the previous changes to the scale and restore hotkeys and also improve a few other descriptions.
@blessedcoolant
Update WEBUIHOTKEYS.md
64dcd67 
Update hotkeys docs. They were outdated.
@blessedcoolant
Fresh Frontend Build
dbd8064
hipsterusername
hipsterusername approved these changes Dec 13, 2022
View reviewed changes
Copy link
Member

@hipsterusername hipsterusername left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@blessedcoolant blessedcoolant merged commit 474220c into invoke-ai:main Dec 13, 2022
@blessedcoolant blessedcoolant deleted the hotkeys-update branch December 13, 2022 07:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@hipsterusername hipsterusername hipsterusername approved these changes

@lstein lstein Awaiting requested review from lstein lstein is a code owner

@psychedelicious psychedelicious Awaiting requested review from psychedelicious

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@blessedcoolant @hipsterusername