Closed Bug 560592 Opened 10 years ago Closed 10 years ago

'switch task' button doesn't give any push feedback when clicking on it

Categories

(Firefox for Android Graveyard :: General, defect)

Fennec 1.1
x86
Windows 7
defect
Not set

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: martijn.martijn, Assigned: mfinkle)

Details

(Keywords: polish)

Attachments

(2 files, 1 obsolete file)

This is what Mark Finkle said in #mobile about it:
<mfinkle>	mw22: we didn't add the "presses" states, iirc
<mfinkle>	"pressed"
mw22: looks like we never added the images for pressed
http://mxr.mozilla.org/mobile-browser/find?string=task-&tree=mobile-browser&hint=
<mfinkle>	those are only for unpressed
<mfinkle>	mw22: please file
<mfinkle>	I guess we could just highlight the background of the buttons
<mfinkle>	n900 actually changes the image too
I think also the close button has this problem. It should also have a blue blackground during the 'mousedown' phase. The same for the 'back' button (the button you get when focusing the url bar).
Keywords: polish
Attached patch patch (obsolete) — Splinter Review
Adds the blue highlight background color when the task switch, app close or back buttons are pressed.

Lightweight, simple CSS patch
Assignee: nobody → mark.finkle
Attachment #440846 - Flags: review?(21)
Attached patch patch 2Splinter Review
This patch expands the buttons to the edges and rounds the inner bottom corner.
Attachment #440846 - Attachment is obsolete: true
Attachment #441078 - Flags: review?(21)
Attachment #440846 - Flags: review?(21)
Attached image screenshot
Screenshot of the button when highlighted
pushed m-b:
http://hg.mozilla.org/mobile-browser/rev/f0c728725029

pushed m-1.1:
http://hg.mozilla.org/releases/mobile-1.1/rev/6107e19ed5ce
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
verified FIXED on builds:

Mozilla/5.0 (X11; U; Linux armv7l; Nokia N900; en-US; rv:1.9.2.5pre) Gecko/20100427 Namoroka/3.6.5pre Fennec/1.1b2pre

and

Mozilla/5.0 (X11; U; Linux armv71; Nokia N900; en-US; rv:1.9.3a5pre) Gecko/20100427 Namoroka/3.7a5pre Fennec/1.1b2pre


... with follow-up bug https://bugzilla.mozilla.org/show_bug.cgi?id=562036
Status: RESOLVED → VERIFIED
Flags: in-litmus?
litmus testcase https://litmus.mozilla.org/show_test.cgi?id=9758 updated to regression test this bug
Flags: in-litmus? → in-litmus+
You need to log in before you can comment on or make changes to this bug.