Closed
Bug 606827
Opened 14 years ago
Closed 14 years ago
Released and collapsing drag regions fade back into... somewhere
Categories
(Firefox Graveyard :: Panorama, defect, P5)
Firefox Graveyard
Panorama
Tracking
(Not tracked)
RESOLVED
FIXED
Firefox 4.0b8
People
(Reporter: mitcho, Assigned: mitcho)
Details
(Whiteboard: [good first bug][visual])
Attachments
(2 files, 2 obsolete files)
44.54 KB,
video/webm
|
Details | |
1.06 KB,
patch
|
Details | Diff | Splinter Review |
When we drag out a region but it's too small so it collapses, it animates like it's fading back to where it came from... but in many cases, that "somewhere" seems to be some weird origin. I propose to make it collapse into itself... into the center of the region. Aza, is this the right spec?
Assignee | ||
Comment 1•14 years ago
|
||
Assignee: nobody → mitcho
Status: NEW → ASSIGNED
Attachment #485622 -
Flags: ui-review?(aza)
Attachment #485622 -
Flags: feedback?(ian)
Comment 2•14 years ago
|
||
Comment on attachment 485622 [details] [diff] [review] Patch v1 Nice catch! All you needed to do was switch the x and y, but this version is cleaner anyway. I'm not in favor of speeding up the animation though... it's already fast enough that you might not get what's happening, and it's not the sort of animation that blocks user action, so there's no real need for it to be fast.
Attachment #485622 -
Flags: feedback?(ian) → feedback+
Assignee | ||
Comment 3•14 years ago
|
||
> I'm not in favor of speeding up the animation though... it's already fast
> enough that you might not get what's happening, and it's not the sort of
> animation that blocks user action, so there's no real need for it to be fast.
Aza, thoughts?
Assignee | ||
Updated•14 years ago
|
Attachment #485622 -
Flags: review?(dolske)
Comment 4•14 years ago
|
||
Agreed. I think the speed is right at the moment and indeed the group should collapse into itself.
Updated•14 years ago
|
Attachment #485622 -
Flags: ui-review?(aza) → ui-review+
Assignee | ||
Comment 5•14 years ago
|
||
Attachment #485622 -
Attachment is obsolete: true
Attachment #485916 -
Flags: review?(dolske)
Attachment #485622 -
Flags: review?(dolske)
Updated•14 years ago
|
Attachment #485916 -
Flags: review?(dolske)
Attachment #485916 -
Flags: review+
Attachment #485916 -
Flags: approval2.0+
Assignee | ||
Comment 6•14 years ago
|
||
Attachment #485916 -
Attachment is obsolete: true
Assignee | ||
Updated•14 years ago
|
Keywords: checkin-needed
Comment 7•14 years ago
|
||
Does it need a try run?
Comment 8•14 years ago
|
||
(In reply to comment #7) > Does it need a try run? Mitcho?
Comment 9•14 years ago
|
||
http://hg.mozilla.org/mozilla-central/rev/18864f1a6b42
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 4.0b8
Updated•8 years ago
|
Product: Firefox → Firefox Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•