Font used for subtext in new Clear Data Dialog is too small
Categories
(Toolkit :: Data Sanitization, defect, P2)
Tracking
()
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
Reporter | ||
Updated•9 months ago
|
Reporter | ||
Comment 1•9 months ago
|
||
Note: the English text is expected, since some strings are still waiting in quarantine.
Reporter | ||
Comment 2•9 months ago
|
||
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.
Updated•9 months ago
|
Assignee | ||
Comment 4•9 months ago
|
||
Depends on D199581
Comment 5•9 months ago
|
||
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
Comment 6•9 months ago
|
||
(In reply to Francesco Lodolo [:flod] from comment #0)
Created attachment 9375623 [details]
Screenshot of Italian BuildThe font is barely readable in Italian (font size shows up as 9.537px).
I assume this is on Mac?
Reporter | ||
Comment 7•9 months ago
|
||
My report is for macOS, but I don't have access to other platforms to confirm if it happens there too.
Assignee | ||
Comment 8•9 months ago
•
|
||
(In reply to Mark Striemer [:mstriemer] from comment #5)
Created attachment 9376383 [details]
with-moz-button-changes.pngHow 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)
Comment 9•9 months ago
|
||
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.
Assignee | ||
Comment 10•9 months ago
|
||
I think we're okay waiting for your patch to land while this dialog is nightly only.
Updated•9 months ago
|
Comment 11•9 months ago
|
||
Should be fixed by bug 1875384, bug 1835869 will further improve this.
Description
•