Closed
Bug 612828
Opened 14 years ago
Closed 14 years ago
Link target and current URL in location bar should transition at same non-linear rate
Categories
(Firefox :: Address Bar, defect)
Firefox
Address Bar
Tracking
()
RESOLVED
FIXED
Firefox 4.0b8
People
(Reporter: adw, Assigned: adw)
References
Details
Attachments
(1 file)
(deleted),
patch
|
dao
:
review+
Gavin
:
approval2.0+
|
Details | Diff | Splinter Review |
When a link target is shown in the location bar and it overlaps the current URL, the current URL fades out linearly, but the target fades in using "ease". Similarly, when a target disappears, it fades out using ease, but the current URL fades in using cubic-bezier(0.0, 0.6, 1.0, 1.0).
It's a little jarring and unpolished. The target and current URL should transition at the same rate. The rate should be non-linear, because linear fades are slow and blah.
Bug 587908 comment 78 and bug 597769 explain why the rates are different -- to avoid flickering the current URL basically.
This patch makes 1) the target fade-in and current URL fade-out rate the same (ease) and 2) the target fade-out and current URL fade-in rate the same (cubic-bezier(0.0, 1.0, 1.0, 1.0)). Rates 1 and 2 are unfortunately still different to avoid flickering the current URL. I chose ease for rate 1 because it's nice, and if rates 1 and 2 have to be different then rate 1 should look nicer than rate 2 IMO. Rate 2 is snappy and seems to avoid flicker just as well as the current rates do.
It also makes 1) the urlbar children (like the bookmark star) fade out the same as the target fade in and 2) the children fade in the same as the target fade out.
Attachment #491121 -
Flags: review?(dao)
Assignee | ||
Comment 1•14 years ago
|
||
(In reply to comment #0)
> Bug 587908 comment 78 and bug 597769 explain why the rates are different -- to
> avoid flickering the current URL basically.
Er, why the two current URL rates are different, not why the target and current URL rates are different.
Updated•14 years ago
|
Attachment #491121 -
Flags: review?(dao) → review+
Assignee | ||
Comment 2•14 years ago
|
||
Comment on attachment 491121 [details] [diff] [review]
patch
Thanks Dão.
Requesting a+ for simple polish to a new Firefox 4 feature.
Attachment #491121 -
Flags: approval2.0?
Updated•14 years ago
|
Attachment #491121 -
Flags: approval2.0? → approval2.0+
Assignee | ||
Comment 3•14 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 4.0b8
You need to log in
before you can comment on or make changes to this bug.
Description
•