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

feat: more shortcuts to switch between chats #3984

Merged
merged 1 commit into from
Jul 1, 2024

Conversation

WofWca
Copy link
Collaborator

@WofWca WofWca commented Jun 27, 2024

Ctrl + PageDown, Ctrl + PageUp,
Ctrl + Tab, Ctrl + Shift + Tab

These are used by Telegram: https://github.com/telegramdesktop/tdesktop/wiki/Keyboard-Shortcuts

Related MR: #3965

I tested it a bit. Appears to work, and not break existing "PageUp" shortcut. Those on Mac, please verify that these shortcuts are alright. see comment about Mac below

image

`Ctrl + PageDown`, `Ctrl + PageUp`,
`Ctrl + Tab`, `Ctrl + Shift + Tab`

These are used by Telegram: https://github.com/telegramdesktop/tdesktop/wiki/Keyboard-Shortcuts

Related MR: #3965
@WofWca WofWca added enhancement New feature or request ui/ux UI/UX related issues accessibility labels Jun 27, 2024
@Simon-Laux
Copy link
Member

I'm not cure about ctrl+tab, we might need it to be a more generic way to navigate through the whole app in the future, also I do not know whether and what key combinations are used or required by screen readers. 🤔

The ones with PageUp and PageDown are fine by me

@Simon-Laux Simon-Laux requested review from nicodh, r10s and farooqkz June 27, 2024 21:49
@WofWca
Copy link
Collaborator Author

WofWca commented Jun 28, 2024

Update about Mac: Telegram has different shortcuts there https://github.com/overtake/TelegramSwift/wiki , only ⌥⌘↑

Update 2: or no?? overtake/TelegramSwift#826, overtake/TelegramSwift#1118 - says that Ctrl + Tab also works (sometimes).

@Simon-Laux
Copy link
Member

just checked linux Firefox and macOS safari. seems like I confused it with alt+tab on macOS and tab & shift+tab everywhere else so its fine.

@Simon-Laux Simon-Laux merged commit 1e11cfd into main Jul 1, 2024
7 checks passed
@Simon-Laux Simon-Laux deleted the wofwca/more-switch-chat-shortcuts branch July 1, 2024 17:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
accessibility enhancement New feature or request ui/ux UI/UX related issues
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants