Closed Bug 1705386 Opened 4 years ago Closed 4 years ago

The text from the "Set Firefox as your default browser" wraps on a second row

Categories

(Toolkit Graveyard :: Notifications and Alerts, defect)

Desktop
Windows 10
defect

Tracking

(firefox87 disabled, firefox88 unaffected, firefox89 verified)

VERIFIED FIXED
89 Branch
Tracking Status
firefox87 --- disabled
firefox88 --- unaffected
firefox89 --- verified

People

(Reporter: rdoghi, Assigned: Gijs)

References

(Blocks 2 open bugs)

Details

(Whiteboard: [proton-modals])

Attachments

(2 files)

Attached image 2021-04-15_14h52_41.png (deleted) —

[Affected platforms]:
Platforms: Windows 10

[Steps to reproduce]

  1. Launch the Firefox browser with a fresh profile.
  2. Close and start Firefox again with the same profile.

[Expected result]
The text from the "Set Firefox as your default browser" modal should not wrap on a second row.

[Actual result]
The text from the "Set Firefox as your default browser" modal wraps on a second row.

Has Regression Range: --- → no
Has STR: --- → yes
Flags: needinfo?(gijskruitbosch+bugs)

Hm, I thought I knew what was happening here but I actually can't reproduce on last night's nightly.

Can you check what build you're on, and what Windows DPI settings you see this at?

Flags: needinfo?(gijskruitbosch+bugs) → needinfo?(rares.doghi)

I was in the latest Nightly, the build from today, I could only reproduce it on Windows 10 and 7, resolution is 1920 x 1080 with the recommended 100% scaling.

Flags: needinfo?(rares.doghi)

This is the build id: 20210414160838

(In reply to Rares Doghi from comment #2)

I was in the latest Nightly, the build from today, I could only reproduce it on Windows 10 and 7, resolution is 1920 x 1080 with the recommended 100% scaling.

Thanks for the super quick response.

This is gonna be funnn, rounding errors! I can repro with 100% scaling, but not with 150% (the default for my machine). Investigating further...

Assignee: nobody → gijskruitbosch+bugs
Status: NEW → ASSIGNED

As @jfkthame pointed out on matrix, 'ch' units are based on the
rendering of the '0' character, and thus their width in CSS 'px'
can vary for fonts that line things up to device pixels, which
can cause pretty big variations if you start multiplying with
large numbers like the dialog width. As a result, the dialog
was 30 CSS pixels narrower at 100dpi than at 150dpi, which
caused the wrapping.

I also noticed we forgot to update the width for checkboxes
and text in dialogs after our font-size changes in bug 1701920,
so I took the opportunity of fixing that here.

Pushed by gijskruitbosch@gmail.com: https://hg.mozilla.org/integration/autoland/rev/ea0c0f99c53d set the correct width on dialogs across scaling, r=jaws
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → 89 Branch

This issue is Verified as fixed in our latest Nightly build 89.0a1 (2021-04-16).

Status: RESOLVED → VERIFIED
Whiteboard: [proton-modals]
Product: Toolkit → Toolkit Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: