Closed
Bug 906873
Opened 11 years ago
Closed 11 years ago
[Clock] On press active state "new alarm" icon is bigger and fuzzy
Categories
(Firefox OS Graveyard :: Gaia::Clock, defect, P1)
Tracking
(blocking-b2g:hd+, b2g-v1.1hd fixed)
People
(Reporter: amylee, Assigned: pivanov)
Details
(Whiteboard: helix vsd, HD)
Attachments
(3 files)
Hi, In the clock app, when you hold down on the "add new alarm" icon, the icon is bigger and fuzzy. Please let me know if you need a new graphic for this fix. Thanks
Reporter | ||
Updated•11 years ago
|
Reporter | ||
Updated•11 years ago
|
Hardware: x86 → All
Reporter | ||
Updated•11 years ago
|
Assignee: nobody → pivanov
Reporter | ||
Updated•11 years ago
|
Status: UNCONFIRMED → NEW
Ever confirmed: true
Assignee | ||
Comment 1•11 years ago
|
||
Hey Eric, we had this issue with the scrubber. We don't have support for background-image transitions and actually this behavior here is unwanted. Do I need to remove the active state?
Flags: needinfo?(epang)
Comment 2•11 years ago
|
||
If we want to keep the active icon, my suggestion is as following. I would like to say the icon of active state is not bigger enough while a user is tapping on it. So that we are able to see a little bit active state after the finger is not blocking our sight. If I don't pay attention, I might not see it.
Comment 3•11 years ago
|
||
(In reply to Pavel Ivanov [:ivanovpavel] from comment #1) > Hey Eric, > we had this issue with the scrubber. We don't have support for > background-image transitions and actually this behavior here is unwanted. > Do I need to remove the active state? Yes IMO i think we can remove the active state. As Ian noted it's not noticeable when our finger is on it anyway. We can look into an active state in the future. Thanks!
Flags: needinfo?(epang)
Assignee | ||
Comment 5•11 years ago
|
||
Attachment #793317 -
Flags: review?(sjochimek)
Assignee | ||
Comment 6•11 years ago
|
||
Updated•11 years ago
|
Whiteboard: helix vsd → helix vsd, HD
Comment 7•11 years ago
|
||
Comment on attachment 793317 [details] patch for Gaia/master Landed in master: https://github.com/mozilla-b2g/gaia/commit/d82a1a61a08499df8b519c51ecddb61d4c2fde18 Landed in v1.1.0hd: https://github.com/mozilla-b2g/gaia/commit/1887ba2b14aa4537e7addd1e24c8abc2205a1574
Attachment #793317 -
Flags: review?(sjochimek) → review+
Updated•11 years ago
|
Comment 8•11 years ago
|
||
Thank you! Verified on following V1.1.0 HD build. * Test build:(Mozilla-b2g18_v1_1_0_hd-helix/2013-08-24-04-22-02) + Mercurial-Information - Gecko revision="f392714dea0b" + Git-information - Gecko revision="a87463bf8387cc5001824a6411f7cd0497bb8649" - Gaia revision="7b61b7dc3400d5bea23d2b52948f4fded8259dfc" Marked as "Verified"
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•