Closed Bug 1007629 Opened 11 years ago Closed 10 years ago

don't use "embedded" fonts for in-content preferences page

Categories

(Firefox :: Settings UI, defect)

x86
macOS
defect
Not set
normal

Tracking

()

VERIFIED FIXED
Firefox 32
Tracking Status
firefox32 --- verified
firefox34 --- verified

People

(Reporter: jfkthame, Assigned: jfkthame)

References

Details

Attachments

(1 file, 1 obsolete file)

about:preferences should be using the host system's standard UI font, not a font embedded in the browser, for a number of reasons: - as part of the browser's user interface, about:preferences should conform to the expected appearance of UI on the host platform; - there's a memory and performance cost to loading the embedded fonts, which is not justified for content that does not really require a specific, custom font but should be conforming to the platform's theme; - the embedded fonts being shipped in the browser are emasculated subsets that do not support the full character set required for all localizations, resulting in poor appearance in some languages (compare bug 992650 re about:accounts); - the rendering of the embedded fonts may be worse than that of standard system fonts that have been carefully hinted for a particular OS font rasterizer; this is especially an issue on older Windows systems under GDI; - using embedded fonts will prevent the page respecting custom themes that a user may have chosen at the OS level. So I propose we should replace the use of "Clear Sans" here with "font:message-box", which will select the appropriate host system font for such content - defaulting to Lucida Grande on OS X, Segoe UI on Win7/8, Tahoma on WinXP, ... (Comparing the two versions of about:preferences on OS X, ISTM that the Lucida Grande version looks *much* more like it belongs on my Mac; the Clear Sans version feels "foreign", with an overall look'n'feel that is quite jarring. Clear Sans is a perfectly nice font, but it is *not* the Mac's user interface font.)
Attachment #8419360 - Flags: review?(gavin.sharp)
Blocks: 1007639
Attachment #8419360 - Flags: review?(gavin.sharp) → review+
Unbitrotted patch, ready for checkin.
Attachment #8419360 - Attachment is obsolete: true
Assignee: nobody → jfkthame
Status: NEW → ASSIGNED
Comment on attachment 8435591 [details] [diff] [review] eliminate use of "embedded" fonts in about:preferences. Carrying over r=gavin.
Attachment #8435591 - Flags: review+
Keywords: checkin-needed
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 32
Keywords: verifyme
QA Contact: camelia.badau
Verified fixed on Mac OSX 10.8.5 using Firefox 32 Beta 4 (buildID: 20140804164216) and latest Nightly 34.0a1 (buildID: 20140806030201).
Status: RESOLVED → VERIFIED
Keywords: verifyme
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: