1655244 - Ctrl+scroll wheel does not zoom in Thunderbird message reader, message pane or compose window

Status: VERIFIED FIXED

(Thunderbird :: Mail Window Front End, defect, P1)

Description

[Magnus Melin [:mkmelin]]

+++ This bug was initially created as a clone of Bug #1634556 +++

Splitting off the Thunderbird part to this bug. See bug 1634556 comment 11:

Thunderbird doesn't use e10s yet. The compose content is in an <editor> https://searchfox.org/comm-central/rev/eb69eff3ca16509af6cf8f2dd29be396bebbc14d/mail/components/compose/content/messengercompose.xhtml#2526 and the viewing content in a <browser> https://searchfox.org/comm-central/rev/eb69eff3ca16509af6cf8f2dd29be396bebbc14d/mail/base/content/messenger.xhtml#769

I see we have a forked ZoomManager. That might need needs adjustments - https://searchfox.org/comm-central/search?q=ZoomManager&path=

If there's a Thunderbird developer who is prepared to do the work, the https://phabricator.services.mozilla.com/D84767 patch shows what needs to be done:

On the parent process, listen for the "DoZoomEnlargeBy10" and ""DoZoomReduceBy10" events, and either set browser.fullZoom (if you have a browser) or docShell.zoom in increments of 0.1.
Cap the zoom levels based on the prefs "zoom.minPercent" and "zoom.maxPercent". The D84767 patch gets these values from ZoomManager, but that's not strictly necessary, and they can be retrieved using more direct means.

Comment 1

[Khushil Mistry [:khushil324]]

Attachment: Bug-1655244_ctrl-scroll-zoom-not-working-0.patch

Comment 2

[Magnus Melin [:mkmelin]]

Comment on attachment 9166386 [details] [diff] [review]
Bug-1655244_ctrl-scroll-zoom-not-working-0.patch

Review of attachment 9166386 [details] [diff] [review]:
-----------------------------------------------------------------

Looks good, thx! r=mkmelin
A small unbitrotting was necessary

Comment 3

[Khushil Mistry [:khushil324]]

Attachment: Bug-1655244_ctrl-scroll-zoom-not-working-1.patch

[Approval Request Comment]
Regression caused by (bug #): 1516413
User impact if declined: User will not be able to zoom in or out with scroll event. It can impact the experience of users who were relying on scroll events to change the zoom while reading the mail.
Testing completed (on c-c, etc.):
Risk to taking this patch (and alternatives if risky): Low.

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/comm-central/rev/9d32224ed772
Fix Ctrl+scroll wheel does not zoom in Thunderbird message reader, message pane or compose window. r=mkmelin

Comment 5

[Wayne Mery (:wsmwk)]

Comment on attachment 9167672 [details] [diff] [review]
Bug-1655244_ctrl-scroll-zoom-not-working-1.patch

[Triage Comment]
Approved for beta

Comment 6

[Rob Lemley [:rjl]]

Thunderbird 80.0b2:
https://hg.mozilla.org/releases/comm-beta/rev/ce3666e957db

Comment 7

[WaltS48 [:walts48]]

Working in my testing of the 80.0b2 release candidate on Ubuntu 18.04.4.

Comment 8

[Wayne Mery (:wsmwk)]

Comment on attachment 9167672 [details] [diff] [review]
Bug-1655244_ctrl-scroll-zoom-not-working-1.patch

[Triage Comment]
Approved for esr78

Comment 9

[Geoff Lankow (:darktrojan)]

Thunderbird 78.1.2:
https://hg.mozilla.org/releases/comm-esr78/rev/cc551f9b30e8

Comment 10

[Rachel Martin]

This should have been fixed for the message source window as well:
https://github.com/Betterbird/thunderbird-patches/blob/main/91/bugs/1655244-ctrl-scroll-source.patch

Comment 11

[Magnus Melin [:mkmelin]]

Filed bug 1749305 for that.

Comment 12

[Magnus Melin [:mkmelin]]

Attachment: Bug 1655244 - Fix Ctrl+scroll wheel zoom in message source window. r=henry

Based on https://github.com/Betterbird/thunderbird-patches/blob/main/91/bugs/1655244-ctrl-scroll-source.patch

Comment 13

[Phabricator Automation]

Comment on attachment 9258333 [details]
Bug 1655244 - Fix Ctrl+scroll wheel zoom in message source window. r=henry

Revision D135511 was moved to bug 1749305. Setting attachment 9258333 [details] to obsolete.