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