Opened 9 years ago

Closed 9 years ago

#12177 closed defect (fixed)

[patch][ccla] wipeIn/Out stop not cleaning up style changes

Reported by: Douglas Hays Owned by: Douglas Hays
Priority: high Milestone: 1.7
Component: fx Version: 1.6.0b1
Keywords: Cc: dante, Eugene Lazutkin
Blocked By: Blocking:

Description

fx.wipeIn and wipeOut both set style.overflow=hidden when they start, and then resets this onEnd. But if stop() is called, the style changes are made permanent. The same style cleanup should also occur onStop. This was found by a customer using dijit.Tree. When a node was double-clicked, it expands and collapses more quickly than the animation and the overflow=hidden remains which truncates the node label.

Attachments (1)

12177.patch​ (2.1 KB) - added by Douglas Hays 9 years ago.
patch for review

Download all attachments as: .zip

Change History (7)

Changed 9 years ago by Douglas Hays

Attachment: 12177.patch​ added

patch for review

comment:1 Changed 9 years ago by Douglas Hays

Cc: dante Eugene Lazutkin added
Summary: wipeIn/Out stop not cleaning up style changes → [patch][ccla] wipeIn/Out stop not cleaning up style changes

I'd like to get the attached patch reviewed and committed for 1.6 if possible. I included an automated test for both wipeIn and wipeOut.

comment:2 Changed 9 years ago by dante

Owner: changed from Bryan Forbes to bryanforbes

comment:3 Changed 9 years ago by bill

Owner: changed from bryanforbes to Bryan Forbes

comment:4 Changed 9 years ago by Douglas Hays

Milestone: 1.6 → 1.7

Any challenges to me committing this patch into trunk (1.7)?

comment:5 Changed 9 years ago by Douglas Hays

Owner: changed from Bryan Forbes to Douglas Hays
Status: new → assigned

comment:6 Changed 9 years ago by Douglas Hays

Resolution: → fixed
Status: assigned → closed

(In [24124]) Fixes #12177. Duplicate cleanup handler for both onStop and onEnd. Added automated test.

Note: See TracTickets for help on using tickets.