Closed Bug 760970 Opened 12 years ago Closed 12 years ago

Custom menu button hard to press after recent redesign

Categories

(Firefox for Android Graveyard :: General, defect)

15 Branch
ARM
Android
defect
Not set
major

Tracking

(firefox15 fixed)

VERIFIED FIXED
Firefox 16
Tracking Status
firefox15 --- fixed

People

(Reporter: micmon, Assigned: sriram)

References

Details

Attachments

(5 files)

In recent builds featuring the redesigned UI, the menu button moved to the left of the new tab button and is very hard to press now. The touch target should be larger. Also, the target seems to be only on the left side of the button right now, as pressing on the middle/right activates the new tab button.
Agreed, running into this too on my Galaxy Nexus (Android 4.0.4). Sriram?
tracking-fennec: --- → ?
OS: Linux → Android
Summary: Menu button hard to press after redesign → Custom menu button hard to press after recent redesign
Attached is a screenshot with visible touch feedback enabled on Android 4.0.4 - the target sampled in the zone in the screenshot is overlaid over the menu button but triggers the 'new tab' button.
More obvious with button-down highlight ...
The width is 36dp on portrait mode. This should be 48dp to match the height of the action bar. Also, it should on top of the tabs-button (with a transparent background), to avoid accidental touches being delivered to tabs button (especially when hitting the curve).
Assignee: nobody → sriram
Version: Firefox 15 → Firefox 16
This patch resizes the menu button to be square, and places it on top of the tabs button. The touch goes to menu button and not to tabs.
Attachment #631272 - Flags: review?(mark.finkle)
This patch optimizes the orange color used. I felt we are using way too many orange highlight, even though they don't come with a texture. The other problem is they are not scalable.

I'll attach a screenshot with differences soon.
Attachment #631274 - Flags: review?(mark.finkle)
Attached image Screenshot: Comparison
This shows the difference between the two.
Basically they both don't have any difference.
Ian's resource has a 1pixel yellow color on the top, that merges with the background's top 1px inside glow color.
Given the fact that this is going to be "under" the finger, and not going to be visible to the user much when he press, I guess we can use a gradient and kill 15 resources in the name of https://areweslimyet.com/
Attachment #631272 - Flags: review?(mark.finkle) → review+
Attachment #631274 - Flags: review?(mark.finkle) → review+
https://hg.mozilla.org/mozilla-central/rev/b79284be046a
https://hg.mozilla.org/mozilla-central/rev/c4b6f8f04f9a

(Merged by Ed Morley)
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 16
I know this is resolved already, but I just wanted to add that I am happy to see a gradient selection applied in code rather than with a bunch of different images. I too am a fan of efficiency. Sometimes. :)
Verified fixed on Nightly 16.0a1 (2012-06-12)
Samsung Galaxy SII (2.3.4)
Status: RESOLVED → VERIFIED
Blocks: 739407
Comment on attachment 631272 [details] [diff] [review]
Patch (1/2): Menu button resize

[Approval Request Comment]
Bug caused by (feature/regressing bug #): 
User impact if declined: Soft Menu button will be hard to press on Nexus.
Testing completed (on m-c, etc.): 
Risk to taking this patch (and alternatives if risky): 
String or UUID changes made by this patch:
Attachment #631272 - Flags: approval-mozilla-aurora?
Comment on attachment 631274 [details] [diff] [review]
Patch (2/2): Optimize the orange color

[Approval Request Comment]
Bug caused by (feature/regressing bug #): 
User impact if declined: More resources will be used unnecessarily.
Testing completed (on m-c, etc.): 
Risk to taking this patch (and alternatives if risky): 
String or UUID changes made by this patch:
Attachment #631274 - Flags: approval-mozilla-aurora?
Blocks: 760090
Blocks: 763851
Comment on attachment 631272 [details] [diff] [review]
Patch (1/2): Menu button resize

[Triage Comment]
Little risk this early in the cycle.
Attachment #631272 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Attachment #631274 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Version: Firefox 16 → Firefox 15
tracking-fennec: ? → ---
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: