Opened 10 years ago

Last modified 3 years ago

#10077 assigned defect

dojox.fx.wipeTo Cannot Wipe to 0 width

Reported by: ole Owned by:
Priority: high Milestone: 1.15
Component: fx Version: 1.3.2
Keywords: Cc: ole.ersoy@…
Blocked By: Blocking:

Description

This does not work:

var currentNodeWipeOutParameters =

{ node: "currentNode", height: 100, width: 0, duration: 300

};

var currentNodeWipeOutAnimation =

dojox.fx.wipeTo(

currentNodeWipeOutParameters);

But this does:

var currentNodeWipeOutParameters =

{ node: "currentNode", height: 100, width: 1, duration: 300

};

var currentNodeWipeOutAnimation =

dojox.fx.wipeTo(

currentNodeWipeOutParameters);

Applied to this node:

<div id="currentNode"

style=" background-color: black;

top: 200px; left: 100px; width: 100px; height: 100px; position: absolute;">

</div>

Change History (13)

comment:1 Changed 10 years ago by Eugene Lazutkin

exact dup of #10078

comment:2 Changed 10 years ago by Eugene Lazutkin

Component: Generalfx
Owner: changed from anonymous to Bryan Forbes

comment:3 Changed 10 years ago by dante

Owner: changed from Bryan Forbes to dante

I'm dojox.fx, not sure if the "fx" label in trac is assigning these to Bryan or not, but it is in fact me. (nic also does a bit of the fx work these days, too)

comment:4 Changed 10 years ago by Eugene Lazutkin

fx assigns to Bryan automatically. I just changed the component.

comment:5 Changed 10 years ago by dante

Milestone: tbd1.5

not seeing why this is apparently. need to dig.

comment:6 Changed 10 years ago by ole

Seems like a division by 0 somewhere...I'm looking as well, but it's a tricky baggar.

comment:7 Changed 10 years ago by dante

actually, while looking at dojo.fx.wipeIn/Out earlier, I noticed we're forcing 1px because animating to 0px height causes issues. (we set display:none after going to 1px in onEnd) so perhaps this is just a limitation and we should change the defaults to
1 instead of 0

comment:8 Changed 10 years ago by ole

A chimp could probably provide more insight than me on that (I still write better tests than a chimp though), but when I wipe to 0 height it works great, so +1 from me.

I noticed that the final state when wiping to 0 height is what one would expect (The final height is 0px and display is block), so I assume the display is set to none temporarily while the height is set to 0 and then set back to what it was originally?

comment:9 Changed 9 years ago by Adam Peller

Milestone: 1.51.6

comment:10 Changed 8 years ago by dante

Milestone: 1.6future

comment:11 Changed 4 years ago by dylan

Milestone: future1.12

comment:12 Changed 4 years ago by dylan

Owner: dante deleted
Status: newassigned

comment:13 Changed 3 years ago by dylan

Milestone: 1.131.15

Ticket planning... move current 1.13 tickets out to 1.15 to make it easier to move tickets into the 1.13 milestone.

Note: See TracTickets for help on using tickets.