Closed Bug 1875642 Opened 9 months ago Closed 9 months ago

Font used for subtext in new Clear Data Dialog is too small

Categories

(Toolkit :: Data Sanitization, defect, P2)

Unspecified
macOS
defect

Tracking

()

RESOLVED FIXED

People

(Reporter: flod, Assigned: hsohaney)

References

(Depends on 1 open bug, Blocks 1 open bug)

Details

(Keywords: priv-next, Whiteboard: fixed by bug 1875384)

Attachments

(3 files, 1 obsolete file)

The font is barely readable in Italian (font size shows up as 9.537px). I expect languages like Chinese, Japanese, or Arabic to have issues at this font dimension.

For context, see also this issue for the tab media notification
https://bugzilla.mozilla.org/show_bug.cgi?id=1699841#c3

Attachment #9375623 - Attachment description: immagine.png → Screenshot of Italian Build
Attachment #9375623 - Attachment filename: immagine.png → italian.png
Attached image Screenshot zh-TW build

Note: the English text is expected, since some strings are still waiting in quarantine.

Feedback from one of our zh-CN translators: on a low-resolution screen the text is basically unrecognizable, on higher resolution it's better but still too small to read easily.

Thanks for reporting this. We'll run it by UX team.

Assignee: nobody → hsohaney
Keywords: priv-next
Priority: -- → P2
Severity: -- → S3

How soon is this supposed to be enabled? This is an issue I ran into as well when updating some buttons (bug 1835869) in similar dialogs and the changes there seem to fix this too

(In reply to Francesco Lodolo [:flod] from comment #0)

Created attachment 9375623 [details]
Screenshot of Italian Build

The font is barely readable in Italian (font size shows up as 9.537px).

I assume this is on Mac?

Depends on: 1875384
OS: Unspecified → macOS

My report is for macOS, but I don't have access to other platforms to confirm if it happens there too.

Depends on: 1835869

(In reply to Mark Striemer [:mstriemer] from comment #5)

Created attachment 9376383 [details]
with-moz-button-changes.png

How soon is this supposed to be enabled? This is an issue I ran into as well when updating some buttons (bug 1835869) in similar dialogs and the changes there seem to fix this too

We were looking to enable the dialog in nightly within the next week.

We've brought it up with UX to see what needs to be done in this case. It could be a bigger problem with the "text-demphasized" class since that is what I used for the description (https://searchfox.org/mozilla-central/source/browser/base/content/sanitize_v2.xhtml#161)

The issue isn't with the text-deemphasized class per se but it's with the base formatting of these dialogs. This wouldn't be the only dialog that is affected.

Basically the dialogs are pretending to be in-content where we use 15px font size and we want text-deemphasized to be smaller, but these dialogs actually override the text to be 11px, which is causing the text-deemphasized class to make the font way too small.

There isn't anything wrong with the text-deemphasized class, just the CSS that is setting up our base font sizes that it is using, that's what my patch in the other bug fixes. The change in there is small enough that if we need this fixed ASAP I can extract it to land without the other changes, but if this is held to Nightly for a bit then we can probably wait for that stack to land as it is nearly ready.

I think we're okay waiting for your patch to land while this dialog is nightly only.

Attachment #9376341 - Attachment is obsolete: true

Should be fixed by bug 1875384, bug 1835869 will further improve this.

Status: NEW → RESOLVED
Closed: 9 months ago
Resolution: --- → FIXED
Whiteboard: fixed by bug 1875384
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: