Closed
Bug 686450
Opened 13 years ago
Closed 13 years ago
Fennec uses wrong DPI on maemo, shows tablet ui
Categories
(Firefox for Android Graveyard :: General, defect)
Tracking
(Not tracked)
VERIFIED
FIXED
Firefox 9
People
(Reporter: heeen, Assigned: heeen)
Details
Attachments
(1 file, 1 obsolete file)
901 bytes,
patch
|
mbrubeck
:
review+
|
Details | Diff | Splinter Review |
Maemo needs a manual override to 240DPI to be usable because its X11 lies about that.
Assignee | ||
Comment 1•13 years ago
|
||
This bug was introduced by changes from bug 658319.
Comment 2•13 years ago
|
||
Comment on attachment 559951 [details] [diff] [review]
override DPI on maemo
> #ifndef MOZ_PLATFORM_MAEMO
> // On desktop builds, simulate an MDPI tablet by default.
> pref("layout.css.dpi", 160);
>+#elsif
>+// Maemo X11 lies about its dpi
>+pref("layout.css.dpi", 240);
> #endif
"#elsif" should be just "#else" here.
Attachment #559951 -
Attachment is patch: true
Assignee | ||
Comment 3•13 years ago
|
||
my bad
Attachment #559951 -
Attachment is obsolete: true
Attachment #559955 -
Flags: review?(mbrubeck)
Updated•13 years ago
|
Attachment #559955 -
Flags: review?(mbrubeck) → review+
Comment 4•13 years ago
|
||
Assignee: nobody → florian.haenel
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 9
Comment 5•13 years ago
|
||
Build ID: Mozilla/5.0 (Maemo; Linux armv7l; rv:9.0.a2) Gecko/20111019 Firefox/9.0a2 Fennec/9.0a2
Device: Nokia N900
In about:config, preference: layout.css.dpi=240
Marking bug VERIFIED
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•