Closed
Bug 463933
Opened 16 years ago
Closed 14 years ago
Default size for system fonts is too small on N810.
Categories
(Core :: Graphics, defect)
Tracking
()
RESOLVED
WONTFIX
People
(Reporter: romaxa, Unassigned)
References
Details
Attachments
(1 file)
(deleted),
patch
|
Details | Diff | Splinter Review |
I think we should have some special property to increase default size for system fonts, without system hacking, and changing DPI value.
Attachment #347187 -
Flags: review?(pavlov)
Comment 1•16 years ago
|
||
This seems broken. Do the system fonts in Gecko fail to look like the system fonts as used in other apps? If so, why? If not, then the bug would be that you've configured the system fonts too small. Is our detection of the system DPI incorrect somehow?
Reporter | ||
Comment 2•16 years ago
|
||
Detection system is mostly correct, but only for desktop screen.
On N810 screen pixels are too small, but DPI the same as on desktop (96) and in such combination XUL dialog's, buttons... font looks too small.
Comment 3•16 years ago
|
||
Do they look the same size as in other apps? What the code you're changing is trying to do is make the fonts the same size as the system fonts. If it's failing, that's a bug in the code, not something we should add a pref for.
That code should still work correctly when mozilla's DPI is configured to something different from the system's DPI.
Assignee | ||
Updated•16 years ago
|
Product: Core → Core Graveyard
Comment 4•16 years ago
|
||
This should not have ended up in core graveyard.
Component: GFX → GFX: Thebes
Product: Core Graveyard → Core
QA Contact: general → thebes
Reporter | ||
Comment 5•14 years ago
|
||
Comment on attachment 347187 [details] [diff] [review]
Proposed preference for increasing default system fonts size
Ok, this bug probably still make sense, but this fix is obsolete.
Attachment #347187 -
Flags: review?(pavlov)
Reporter | ||
Comment 6•14 years ago
|
||
Stuart do we have any other bug, or way to manage default system fonts size for different mobile platforms? does it still make sens do anything in this bug? or we can just close it as wontfix?
Comment 7•14 years ago
|
||
n810 isn't supported.
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → WONTFIX
You need to log in
before you can comment on or make changes to this bug.
Description
•