Closed Bug 1585559 Opened 6 years ago Closed 4 years ago

Multi-message view can't be zoomed by ctrl + mousewheel

Categories

(Thunderbird :: Message Reader UI, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
91 Branch

People

(Reporter: standard8, Assigned: standard8)

Details

Attachments

(1 file)

STR

  1. Open a single message
  2. Use ctrl + mousewheel

=> Message zooms.

  1. Open multiple messages so you get the thread summary.
  2. Use ctrl + mousewheel
  • Expected: The zoom is affected
  • Actual: No change.

This works if you use ctrl/cmd + or - (this was originally enabled in bug 495922).

Mark, any idea where we need to hook that up?

Note: I have this effect/defect on both windows and linux environments.Even Ctrl+ anld Ctr - are not always reliable

(In reply to Jorg K (GMT+2) from comment #1)

Mark, any idea where we need to hook that up?

No idea unfortunately. I think the mouse wheel might be hooked up in the backend somewhere. I did wonder if primary=true which is set on the single view & not the multi-message view might be affecting it, but my brief experiment didn't seem to fix it - though my experiment might have been wrong and needed actual code written.

Summary: Mulit-message view can't be zoomed by ctrl + mousewheel → Multi-message view can't be zoomed by ctrl + mousewheel

I've just strayed upon the fix whilst looking at something else.

Assignee: nobody → standard8
Status: NEW → ASSIGNED

Just for info: the issue is still open/visible in 78.11.0

(In reply to Thommie Rother from comment #5)

Just for info: the issue is still open/visible in 78.11.0

Yes, that's why this hasn't been marked as fixed, and that I said that I just found the fix for it.

Nice, this will put some zoom functions in BrowseInTab out of business. It would be extra nice to implement ctrl-mousewheel in all other tab types - contentTab (about pages, web pages), chromeTab, and gloda and chat and calendar types.

Pushed by mbanner@mozilla.com:
https://hg.mozilla.org/comm-central/rev/eea2f34db4ed
Allow zooming the multimessage view with ctrl+mousewheel. r=mkmelin

Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → 91 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: