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

Ctrl+Keypad Plus seem not to work #1325

Closed
Glog78 opened this issue Oct 30, 2023 · 8 comments · Fixed by #1328 or #1581
Closed

Ctrl+Keypad Plus seem not to work #1325

Glog78 opened this issue Oct 30, 2023 · 8 comments · Fixed by #1328 or #1581
Labels
bug Something isn't working
Milestone

Comments

@Glog78
Copy link

Glog78 commented Oct 30, 2023

Contour Terminal version

0.4.0.r4132.31248d53

Installer source

Github: source code cloned

Operating System

Arch

Architecture

x86-64

Other Software

none

Steps to reproduce

I have the following lines in my input mapping:
- { mods: [Control], key: Plus, action: IncreaseFontSize }
- { mods: [Control], key: Minus, action: DecreaseFontSize }

Press ctrl + (Numpad) Plus or ctrl + (Numpad) Minus

Expected Behavior

Resizing the Font Size

Actual Behavior

Nothing

Additional notes

Works with the regular Minus and Plus and worked with the last version i used.

@Glog78 Glog78 added the bug Something isn't working label Oct 30, 2023
@christianparpart
Copy link
Member

nice, finaally somebody that actually uses the keypad. Was this ever working? When introducing CSIu keyboard protocol, I noticed that we didn't really properly implement keypad encoding, and that in the Qt frontend side of life, we didn't even have the mappings for it. So (post 0.3.12 release) I killed it, and just revived the backend-side of it just yesterday night (#1324). Looks like I need to further invest in getting the frontend side working (again), as well.

- { mods: [Control], key: Plus, action: IncreaseFontSize }
- { mods: [Control], key: Minus, action: DecreaseFontSize }

ok, lemme get a full-sized keyboard, to repro and finish the implementation then. :)

@christianparpart christianparpart added this to the 0.4.0 milestone Oct 31, 2023
@Glog78
Copy link
Author

Glog78 commented Oct 31, 2023

Thank you for looking into it. Yes it was working but please don't ask me which version of 0.3.x.

@christianparpart
Copy link
Member

@Glog78 it was working in the past, because we had no explicit handling of numpad, which we now do, because we recently introduced CSIu support (improved keyboard handling for terminal apps, like nvim). This caused numpad to magically stop working for input bindings. I've also fixed that in the related PR (#1328) and should be working soon.

I'm writing this, because now the config is actually strict. + on the main key area is not the same as the + on the numpad. Use Numpad_Add for that (like the documentation in the default contour.yml also documented. Now we adhere to what we document).

Thanks for your patience. :)

@Glog78
Copy link
Author

Glog78 commented Nov 10, 2023

Thanks for fixing :) works now

@Glog78
Copy link
Author

Glog78 commented Jul 29, 2024

btw there seems to be a regression ... at least in current versions and i dunno since when version the numpad doesn't work again

@Yaraslaut
Copy link
Member

I will reopen it, just in case so we will not forget about it
also, new release is coming soon, you can check if this is fixed on master, if not, we will investigate further

@Yaraslaut Yaraslaut reopened this Jul 29, 2024
@Glog78
Copy link
Author

Glog78 commented Jul 30, 2024

Contour Terminal Emulator 0.4.4-master-34289b23

- { mods: [Control],        key: Numpad_Add,     action: IncreaseFontSize }
- { mods: [Control],        key: Numpad_Plus,    action: IncreaseFontSize }
- { mods: [Control],        key: Plus,           action: IncreaseFontSize }
- { mods: [Control],        mouse: WheelUp,      action: IncreaseFontSize }
- { mods: [Control],        key: Numpad_Subtract,action: DecreaseFontSize }
- { mods: [Control],        key: Numpad_Minus,   action: DecreaseFontSize }
- { mods: [Control],        key: Minus,          action: DecreaseFontSize }
- { mods: [Control],        mouse: WheelDown,    action: DecreaseFontSize }

Numpad doesn't work.

@Yaraslaut
Copy link
Member

@Glog78 will be fixed on master soon

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
3 participants