Closed
Bug 908113
Opened 11 years ago
Closed 11 years ago
[UX] Remove back button background fade in
Categories
(Firefox OS Graveyard :: Gaia, defect)
Tracking
(Not tracked)
RESOLVED
DUPLICATE
of bug 821673
People
(Reporter: janjongboom, Assigned: janjongboom)
Details
Attachments
(1 file)
After talking with Josh Carpenter we want to remove the fade in animation on the back button to improve responsiveness.
Assignee | ||
Updated•11 years ago
|
Assignee: nobody → janjongboom
Assignee | ||
Comment 1•11 years ago
|
||
Hi Josh, Here's some changes. Goal was mainly to remove all 'in'-animations to have immediate feedback. Best is to just try it on phone, and have it side by side with a non-patched phone, f.e. SMS app feels more responsive (even though it's not of course).
Attachment #793941 -
Flags: review?(jcarpenter)
Assignee | ||
Comment 2•11 years ago
|
||
Oh here's a video in case: http://www.youtube.com/watch?v=pKjH_l9xexg&feature=em-upload_owner. Left is patched, right is unpatched. Can't see it very good tho.
Comment 3•11 years ago
|
||
Thanks Jan. It's a bit hard to see the result from the video, so we'll flash a device tomorrow (UX team is gathering in SF for a work week) and follow up.
Hey Jan, So I checked out the patch and it's looking really good to me. It certainly helps with the perceived response and I feel that the change is a improvement from what we have. My only reservations with this patch are that the blue "active" state sticks out a little too much now. This will be something that visual design would need to address.
Assignee | ||
Comment 5•11 years ago
|
||
Bueno! You / anyone can play around with that by lowering this value: https://github.com/mozilla-b2g/gaia/pull/11694/files#L0R154 (or maybe changing ease-in into step-end as well)...
Assignee | ||
Comment 6•11 years ago
|
||
Hi Casey, has this been put on some visual design whiteboard?
Flags: needinfo?(kyee)
Jan, I have not only lowered transition time as you were suggesting, but removed the transition, as no other button is animated. Thanks for pointing that out! Refer to: https://bugzilla.mozilla.org/show_bug.cgi?id=821673
Status: NEW → RESOLVED
Closed: 11 years ago
Flags: needinfo?(kyee)
Resolution: --- → DUPLICATE
Updated•11 years ago
|
Attachment mime type: text/plain → text/x-github-pull-request
Assignee | ||
Updated•11 years ago
|
Attachment #793941 -
Flags: review?(jcarpenter)
You need to log in
before you can comment on or make changes to this bug.
Description
•