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)

defect

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)

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?
Attached patch Patch v1 (obsolete) — Splinter Review
Assignee: nobody → mitcho
Status: NEW → ASSIGNED
Attachment #485622 - Flags: ui-review?(aza)
Attachment #485622 - Flags: feedback?(ian)
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+
> 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?
Attachment #485622 - Flags: review?(dolske)
Agreed. I think the speed is right at the moment and indeed the group should collapse into itself.
Attachment #485622 - Flags: ui-review?(aza) → ui-review+
Attachment #485622 - Attachment is obsolete: true
Attachment #485916 - Flags: review?(dolske)
Attachment #485622 - Flags: review?(dolske)
Attachment #485916 - Flags: review?(dolske)
Attachment #485916 - Flags: review+
Attachment #485916 - Flags: approval2.0+
Attachment #485916 - Attachment is obsolete: true
Does it need a try run?
(In reply to comment #7)
> Does it need a try run?

Mitcho?
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
Product: Firefox → Firefox Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: