Feature request: zoom font size with <ctrl> + mouse-wheel about kitty HOT 10 CLOSED

Thanks for reporting

from kitty.

Please @cyd01 consider this enhancement!

Personally I use this combination of < CTRL > + < Mouse Wheel > to change the size with Chrome and Windows (for Icons in the desktop). So I found it very useful.

from kitty.

Hi @cyd01 ,

To integrate this functionality ( < CTRL > + < Mouse Wheel > equivalent to the Menu "Font Up"/"Font Down") you only need to insert at this point:

This code from https://github.com/FauxFaux/PuTTYTray/blob/9dc8d3a636082ba4dbb48de71e59c8e2b9a0e593/windows/window.c#L3879

          // region tray-zoom

        } else if (control_pressed) {
          conf_get_fontspec(conf, CONF_font)->height +=
              MBT_WHEEL_UP == b ? 1 : -1;
          // short version of IDM_RECONF's reconfig:
          term_size(term,
                    conf_get_int(conf, CONF_height),
                    conf_get_int(conf, CONF_width),
                    conf_get_int(conf, CONF_savelines));
          reset_window(2);

          // endregion

        } else {

I hope you'll want to integrate it. It's quite simple and very powerful. 😉
Regards.

from kitty.

Get the very last beta build.
Excuse me in advance ... I did not make so many tests ... I'm about to fly for Christmas holidays.
See you next year.

from kitty.

Get the very last beta build.

Thank you for the trial! However, it doesn't work... because I don't finished to see the code:

• When using your patches, you need to call the function ChangeFontSize() from:
  

  KiTTY/kitty.c

  Line 4530 in 71d79d4

  void ChangeFontSize(HWND hwnd, int dec) {

from kitty.

So the patch will be something like:

          // region tray-zoom

        } else if (control_pressed) {
          ChangeFontSize(hwnd, MBT_WHEEL_UP == b ? 1 : -1);

          // endregion

        } else {

from kitty.

from kitty.

But it is not really beautiful. Font resizing is not so reactive, but I'll do it.

The current menu has this functionality: "Font UP" and "Font DOWN". It's already inside the KiTTY. So this is only another way to call it (more simple and fast, from my point of view).

Thank you for this very good project! 👍

from kitty.

Hi @cyd01 ,

This function works like a charm in the last version 0.73.1.5.
Thank you! 👍

Please, remember to close this issue as it's resolved.
Regards.

from kitty.

Thanks a lot for integrating this, @lars18th and @cyd01!

from kitty.