Closed Bug 625513 Opened 9 years ago Closed 9 years ago

Cleanup CSS problems from using physical units

Categories

(Firefox for Android Graveyard :: General, defect)

x86_64
Linux
defect
Not set

Tracking

(fennec2.0+)

VERIFIED FIXED
Tracking Status
fennec 2.0+ ---

People

(Reporter: mfinkle, Assigned: mfinkle)

References

Details

(Keywords: polish, Whiteboard: [fennec-softblocker])

Attachments

(2 files)

Bug 590817 landed a patch the use physical units in CSS (mozmm) for Android. There are some things we need to cleanup when running on mdpi devices:
* urlbar endcaps have white backgrounds
* browser-controls buttons are too crowded
* dropmarker is stretched in menulist controls
* the thumbnail is slightly offset in it's frame
tracking-fennec: --- → ?
Keywords: polish
tracking-fennec: ? → 2.0+
Assignee: nobody → mark.finkle
Whiteboard: [fennec-softblocker]
Found some others:
* Firefox Spark button is misaligned
* Give Feedback and Get Help on home page are re-flowed poorly
* The "stop" button image is too big and forces the urlbar to grow vertically while loading a page
Attached patch patch 1Splinter Review
This patch fixes:
* Endcaps background
* Firefox Spark (removes the "play" button)
* Fixes the get help and give feedback flow problems
* Removes dead transparent padding in the stop button image to get to 30px

I don't think we will get smaller button images for this release, so I am asking for a review on this patch alone
Attachment #515199 - Flags: review?(mbrubeck)
Attachment #515199 - Flags: review?(mbrubeck) → review+
Attached patch patch 2Splinter Review
While testing this, I found another problem.  At 300dpi or higher, the urlbar and endcap backgrounds are broken.  This fixes them without changing the appearance at lower resolutions, though we need new assets to properly fix the "active" backgrounds.

The only 300dpi Android device available now is the Sharp IS03 in Japan, but you can test this change by setting the "css.layout.dpi" pref to 300.  An IS03 user mentioned problems in a Market review, which I think are solved by this patch.
Attachment #515231 - Flags: review?(mark.finkle)
Attachment #515231 - Flags: review?(mark.finkle) → review+
Matt can close this when he lands. We can file a new bug for 2.0next cleanup
http://hg.mozilla.org/mobile-browser/rev/527ce2a1d1c1
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Verified fixed w/
Mozilla/5.0 (Android; Linux armv7l; rv:2.0b13pre) Gecko/20110317 Firefox/4.0b13pre Fennec/4.0b6pre
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.