Content font size setting affects various UI elements too, breaking the layout

RESOLVED FIXED in 1.6

Status

Instantbird
Conversation
RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: aleth, Assigned: aleth)

Tracking

trunk
x86
Mac OS X

Details

Attachments

(2 attachments, 1 obsolete attachment)

(Assignee)

Description

3 years ago
Created attachment 8531365 [details]
Screen Shot

Affects conv-top in conversations as well as the contact list.
STR:
1) Change Preferences->Content font size. This only appears to affect the conversation content, but...
2) Restart.
(Assignee)

Updated

3 years ago
Duplicate of this bug: 978057
(Assignee)

Comment 2

3 years ago
The font size setting also affects the text in the new tab filterbar, and all the new tab entries. So it probably affects everything with relative font sizes (font-size: medium and the like).
Summary: Content font size setting affects displayName in the UI, breaking the layout → Content font size setting affects various UI elements too, breaking the layout
(Assignee)

Comment 3

3 years ago
(In reply to aleth [:aleth] from comment #2)
> The font size setting also affects the text in the new tab filterbar, and
> all the new tab entries. So it probably affects everything with relative
> font sizes (font-size: medium and the like).

To clarify, those are called "absolute sizes" in the spec (as opposed to "larger" and "smaller") but they are in fact relative to the user's default font size, which determines "medium".
(Assignee)

Comment 4

3 years ago
Created attachment 8531558 [details] [diff] [review]
fontsizes.diff

This patch should make no visible changes if you are using the default font size setting of 13px.
Assignee: nobody → aleth
Status: NEW → ASSIGNED
Attachment #8531558 - Flags: review?(clokep)
(Assignee)

Comment 5

3 years ago
Created attachment 8531632 [details] [diff] [review]
fontsizes.diff v2

Missed one.
Attachment #8531558 - Attachment is obsolete: true
Attachment #8531558 - Flags: review?(clokep)
Attachment #8531632 - Flags: review?(clokep)
Attachment #8531632 - Flags: review?(clokep) → review+
(Assignee)

Updated

3 years ago
Keywords: checkin-needed
(Assignee)

Comment 6

3 years ago
https://hg.mozilla.org/comm-central/rev/1760b72458b3
(Assignee)

Updated

3 years ago
Status: ASSIGNED → RESOLVED
Last Resolved: 3 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → 1.6
You need to log in before you can comment on or make changes to this bug.