After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 760092 - Text Size plugin: Ctrl + Scrollwheel (mouse wheel) moves text instead of just zooming
Text Size plugin: Ctrl + Scrollwheel (mouse wheel) moves text instead of just...
Status: RESOLVED FIXED
Product: gedit-plugins
Classification: Other
Component: General
3.22.x
Other All
: Normal normal
: ---
Assigned To: George White
Gedit maintainers
Depends on:
Blocks:
 
 
Reported: 2016-01-02 23:35 UTC by Iwan Gabovitch
Modified: 2019-03-23 20:34 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Fixed bug #760092 - Text Size plugin: Ctrl + Scrollwheel (mouse wheel) moves text instead of just zooming (1.29 KB, patch)
2017-08-04 08:33 UTC, George White
none Details | Review
Fixed bug #760092 - Text Size plugin: Ctrl + Scrollwheel (mouse wheel) moves text instead of just zooming (1.34 KB, patch)
2017-08-04 14:18 UTC, George White
none Details | Review

Description Iwan Gabovitch 2016-01-02 23:35:06 UTC
Steps to reproduce:
1. start gedit
2. enable text size plugin
3. open text file with 300 lines at 1280x720 window size
4. While line 1 is visible, hold down Ctrl and use the mousewheel downwards

Expected:
Fonts grow bigger, equal to as if you had pressed Ctrl + +

Actual:
Fonts grow bigger but also the scroll is being interpreted as movement

Video of issue:
https://youtu.be/iZjTjwSP7RU

Additional note: In many pieces of software, pressing Shift in "Ctrl + Shift + =" to get "Ctrl + +" is optional and they allow pressing "Ctrl + Shift + -" to get "Ctrl + -". For example: Chrome.
Comment 1 Sébastien Wilmet 2016-01-03 15:41:01 UTC
I confirm the bug, except that when I move the mousewheel downward, the text shrinks, not becomes bigger (but it's maybe a difference in the mousewheel config).
Comment 2 George White 2017-08-04 08:33:43 UTC
Created attachment 356922 [details] [review]
Fixed bug #760092 - Text Size plugin: Ctrl + Scrollwheel (mouse wheel) moves text instead of just zooming

The textsize plugin now tries to keep the currently central line in the
middle.
Comment 3 Sébastien Wilmet 2017-08-04 10:34:25 UTC
Review of attachment 356922 [details] [review]:

The code looks good, if it works.

The commit message should follow these guidelines:
https://wiki.gnome.org/Git/CommitMessages

So the full URL to the bug at the end.
Comment 4 George White 2017-08-04 14:18:24 UTC
Created attachment 356966 [details] [review]
Fixed bug #760092 - Text Size plugin: Ctrl + Scrollwheel (mouse wheel) moves text instead of just zooming

The textsize plugin now tries to keep the currently central line in the
middle.