warpdotdev / Warp

Warp is a modern, Rust-based terminal with AI built in so you and your team can build great software, faster.
https://warp.dev
Other
20.97k stars 359 forks source link

Global hotkey UI should not show/ask different shortcuts for quake mode and show/hide all windows. #3470

Open assayire opened 1 year ago

assayire commented 1 year ago

Discord username (optional)

No response

Describe the bug

The user interface for setting global hotkey is a bit tripping. While the global hotkey currently is intended to be set either for the quake mode window or show/hide all windows, the user interface allows settings different hotkeys for each of them. That gives an impression that you would be able to activate either of the hotkeys which is not the case.

To reproduce

  1. Go to settings > global hotkey.
  2. Choose Dedicated hotkey window. See option to set keyboard shortcut.
  3. Now choose Show/hide all windows. See option to set keyboard shortcut. And you can have different shortcuts while only one of the above options can be chosen.

Expected behavior

Unless Warp supports activating either of the shortcuts / options, the UI should ask only one KB shortcut. One could argue that it is in fact handy to have different shortcuts for each option, the UI will be a bit confusing to new users and even existing ones who have not used this feature before. In practice, users will choose one or other option and stay with it. So UI to configure a single shortcut makes sense currently.

Screenshots

warp

Operating system

MacOS

Operating system and version

Ventura 13.5

Shell Version

No response

Current Warp version

No response

Regression

No, this bug or issue has existed throughout my experience using Warp

Recent working Warp date

No response

Additional context

No response

Does this block you from using Warp daily?

No

Is this a Warp specific issue? (i.e. does it happen in Terminal, iTerm, Kitty, etc.)

N/A, the issue only applies to a Warp specific feature. (i.e. Warp AI, Workflows)

Warp Internal (ignore): linear-label:b8107fdf-ba31-488d-b103-d271c89cac3e

None

warpdotdev-devx[bot] commented 1 year ago

Thanks for letting us know your feedback. This isn't necessarily a bug but I see how it may be confusing to new users and can possibly be improved upon. We'll post any updates on this thread.

To anyone else facing this issue, please add a 👍 to the original post at the top or comment with your details, and subscribe if you'd like to be notified.