settings: expose message_zoom to control web view zoom factor#131
Open
klement wants to merge 1 commit intoakissinger:masterfrom
Open
settings: expose message_zoom to control web view zoom factor#131klement wants to merge 1 commit intoakissinger:masterfrom
klement wants to merge 1 commit intoakissinger:masterfrom
Conversation
The message body and compose panels use QWebEngineView, which renders CSS pt sizes larger than QFont point sizes in native Qt widgets like QTreeView — especially on macOS Retina displays. This causes the message body to appear noticeably larger than the thread list and header panels even when font sizes match. Extract the hardcoded setZoomFactor(1.2) into a configurable message_zoom setting so users can compensate for this discrepancy. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
laarmen
requested changes
Apr 10, 2026
Collaborator
laarmen
left a comment
laarmen
left a comment
There was a problem hiding this comment.
Good idea, however I think the documentation needs some work.
| Retina displays. This means the message body appears bigger than the thread | ||
| list and header panels even when the configured font sizes match. Adjust | ||
| this factor to compensate — values below 1.0 shrink the web view to better | ||
| match the surrounding Qt widgets. |
Collaborator
There was a problem hiding this comment.
issue: most of this isn't relevant for documenting a config knob.
The user doesn't care about QTreeView vs QWebEngineView, they just want to know what will change when they turn this knob in a given direction — that would be the last sentence, except for web view being an implementation detail. You might want to mention instead that this is useful for HTML payloads since they don't necessarily take message_font_size into account.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Adds a
message_zoomsetting (default1.0) that lets users scale the zoom level of the HTML message viewer and the compose view.This is useful on HiDPI displays or when the system font size differs from what the web engine assumes.
Usage in
config.py:Test plan
message_zoom = 1.5and verify message text is larger in thread and compose views