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

Send backtab as a key #1579

Merged
merged 1 commit into from
Aug 7, 2024

Conversation

Yaraslaut
Copy link
Member

Closes #1578
Shift+Tab is Qt::Key_Backtab (https://doc.qt.io/qt-5/qt.html#Key-enum) we should send it as a key, not as a character

@github-actions github-actions bot added the frontend Contour Terminal Emulator (GUI frontend) label Jul 29, 2024
@Yaraslaut Yaraslaut force-pushed the fix/difference_between_backtab_and_tab branch 3 times, most recently from 38479d9 to eede9b1 Compare July 29, 2024 11:17
@Yaraslaut Yaraslaut force-pushed the fix/difference_between_backtab_and_tab branch from eede9b1 to fab81f4 Compare August 5, 2024 15:02
Copy link
Member

@christianparpart christianparpart left a comment

Choose a reason for hiding this comment

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

Thank you, @Yaraslaut :)

for (auto const& mapping: NamedChars)
if (lowerName == mapping.first)
if (upperName == mapping.first)
Copy link
Member

Choose a reason for hiding this comment

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

lol :D
my bad :)

@@ -383,7 +383,7 @@ bool sendKeyEvent(QKeyEvent* event, vtbackend::KeyboardEventType eventType, Term

if (key == Qt::Key_Backtab)
{
session.sendCharEvent(U'\t', physicalKey, modifiers.with(Modifier::Shift), eventType, now);
session.sendKeyEvent(Key::Tab, modifiers.with(Modifier::Shift), eventType, now);
Copy link
Member

Choose a reason for hiding this comment

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

nice catch

@christianparpart christianparpart merged commit b1bf68b into master Aug 7, 2024
39 checks passed
@christianparpart christianparpart deleted the fix/difference_between_backtab_and_tab branch August 7, 2024 05:27
@T-727 T-727 mentioned this pull request Dec 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
frontend Contour Terminal Emulator (GUI frontend)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Key mapping Ctrl+Shift+Tab not working.
2 participants