Closed
Bug 518470
Opened 15 years ago
Closed 15 years ago
Reduce button, toggle and row heights
Categories
(Firefox for Android Graveyard :: General, defect)
Tracking
(Not tracked)
VERIFIED
FIXED
People
(Reporter: mfinkle, Assigned: mfinkle)
Details
(Whiteboard: [polish])
Attachments
(1 file)
7.11 KB,
patch
|
Gavin
:
review+
|
Details | Diff | Splinter Review |
In an effort to maximize the content on the screen, we are attempting to reduce the height of some elements while still making sure they are touch-friendly. This patch: * Changes row height to ~6mm everywhere (awesomebar, bookmark list, prefs, etc.) * Changes button and toggle height to ~4mm * Changes toolbarbutton height to ~5mm (we have very few of these) * Reduces the padding in the search bar and the url bar by half ** search bar is now ~6mm ** urlbar is now ~7mm
Attachment #402477 -
Flags: review?(gavin.sharp)
Updated•15 years ago
|
Attachment #402477 -
Flags: review?(gavin.sharp) → review+
Assignee | ||
Comment 1•15 years ago
|
||
pushed: https://hg.mozilla.org/mobile-browser/rev/867a2dac0998
Status: NEW → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
Updated•15 years ago
|
Whiteboard: [polish]
Comment 2•15 years ago
|
||
I have concerns about reducing general row height to 6mm given that the guidelines for general-population touchability are for 7mm. Buttons can certainly be much less tall, though other platforms increase the size of their invisible touch area (i.e. "iceberg" buttons on the iphone).
Comment 3•15 years ago
|
||
I think this is a candidate for a unit test to be created in order to verify this bug.
Flags: in-testsuite?
Comment 4•15 years ago
|
||
Having tested this on-device I think it's worth using for b4 and getting more feedback about the 6mm row height. The top and bottom bar fixes are good -- the shorter height works fine at the screen edges.
Comment 5•15 years ago
|
||
verified FIXED on build: Mozilla/5.0 (X11; U; Linux armv7l; en-US; rv:1.9.2b1pre) Gecko/20090928 Fennec/1.0b4pre
Status: RESOLVED → VERIFIED
Updated•14 years ago
|
Component: Linux/Maemo → General
OS: Linux → Linux (embedded)
QA Contact: maemo-linux → general
Hardware: x86 → ARM
You need to log in
before you can comment on or make changes to this bug.
Description
•