contour-terminal / contour

Modern C++ Terminal Emulator
http://contour-terminal.org/
Apache License 2.0
2.37k stars 102 forks source link

Ctrl+Keypad Plus seem not to work #1325

Closed Glog78 closed 2 weeks ago

Glog78 commented 9 months ago

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:

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.

christianparpart commented 9 months ago

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. :)

Glog78 commented 9 months ago

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

christianparpart commented 9 months ago

@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 commented 9 months ago

Thanks for fixing :) works now

Glog78 commented 3 weeks ago

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 commented 3 weeks ago

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

Glog78 commented 3 weeks ago

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 commented 3 weeks ago

@Glog78 will be fixed on master soon