Possibly better handling of heights in dojo.fx.wipeIn

[bob.remeika@…]

When using dojo.fx.wipeIn the effect tends to "snap" at the end in our application. I tracked the problem down to a height = "auto" call when the effect ends. When I apply the following diff, it appears to fix the problem in FF3 and Safari 3 both Mac. My fix is just a quick hack, I assume that there might be a better way to handle this.

Here is the diff:

Index: public/javascripts/dojo/dojo/fx.js
===================================================================
--- public/javascripts/dojo/dojo/fx.js  (revision 5771)
+++ public/javascripts/dojo/dojo/fx.js  (working copy)
@@ -336,10 +336,6 @@
                }
        }, args));
 
-       dojo.connect(anim, "onEnd", function(){ 
-               s.height = "auto";
-       });
-
        return anim; // dojo._Animation
 }

[bill]

I think we intentionally set height=auto because (for example) 10 minutes after you wipeIn() a <div>, it's height may change, like for example if the user hits Ctrl+ to increase the font-size, or maybe the content inside the <div> changes, like a setContent() call on a ContentPane.

Of course in many cases the thing being wiped in will never change size (ex: an image).

I wonder setting height=auto causes the jumpy effect in the first place though. Presumably it isn't changing the height... maybe something with margin or borders? Do you have a simple test case?

[bill]

mark all (open) tickets w/blank milestones to be "tbd"; their milestones need to be set to a version number or to "future"

[dylan]

Given the lack of interest over the past 5+ years, I'm closing as patchwelcome. Please re-open if you want to create a pull request.