browser.ui.zoom.pageFitGranularity preference should be 9

RESOLVED FIXED

Status

Firefox for Android Graveyard
General
RESOLVED FIXED
8 years ago
8 years ago

People

(Reporter: Jan Arne Petersen, Assigned: Jan Arne Petersen)

Tracking

(Blocks: 1 bug)

Details

Attachments

(1 attachment)

(Assignee)

Description

8 years ago
browser.ui.zoom.pageFitGranularity preference should be tweaked for Meego to prevent some webpages from downscaling by default.
(Assignee)

Comment 1

8 years ago
Created attachment 468017 [details] [diff] [review]
Set pageFitGranularity to 9 instead of 10 on MOZ_PLATFORM_MAEMO
Attachment #468017 - Flags: review?(mark.finkle)
I wonder if this is really maemo-only. Do you some a few websites that work better if we pagefit them?
(Assignee)

Comment 3

8 years ago
Without that patch youtube is always scaled down to 0.9 zoom level and no block zooming to 1. zoom level is possible. This makes youtube website not so good looking and also slow (because of the zooming) in software rendering case.
Matt, any thoughts on the affect of making this an ALL platform change?
Also, should we consider making a portrait vs. landscape setting?
(In reply to comment #4)
> Matt, any thoughts on the affect of making this an ALL platform change?

If it makes sense for MeeGo, then it makes sense everywhere.  The change seems reasonable to me; the initial 1/10 threshold was arbitrary.

I don't think portrait is important for this preference - very few sites will be close to 100% zoom in portrait, so it doesn't really matter what the threshold is.
OS: Linux → All
Hardware: x86_64 → All
Summary: browser.ui.zoom.pageFitGranularity preference should be 9 for Meego → browser.ui.zoom.pageFitGranularity preference should be 9
Comment on attachment 468017 [details] [diff] [review]
Set pageFitGranularity to 9 instead of 10 on MOZ_PLATFORM_MAEMO

r+ but drop the MAEMO only preprocessor defines.

I can make the change when landing
Attachment #468017 - Flags: review?(mark.finkle) → review+
pushed:
http://hg.mozilla.org/mobile-browser/rev/d7cac9d3ecd8
Status: NEW → RESOLVED
Last Resolved: 8 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.