Opened 11 years ago
Closed 10 years ago
#11457 closed bug (fixed)
Another slideToggle jump effect
Reported by: | anonymous | Owned by: | gnarf |
---|---|---|---|
Priority: | low | Milestone: | 1.8 |
Component: | css | Version: | 1.7.1 |
Keywords: | Cc: | ||
Blocked by: | Blocking: |
Description
A div contains a UL with LIs floated left and with percentage widths. When slideToggle shows the div, sometimes there's a jump at the end of the animation.
This jsfiddle has 12 examples demonstrating the effect.
Giving the UL a width removes the jump, possibly making this a duplicate of #8370 and #9513. And the behavior is similar to #10662.
But it appears if the LI widths add up to 100%, the animation is smooth, which can be seen in the first and second examples.
And when the LI widths don't add up to 100%, there's a jump at the end of the animation.
And confounding things further, in the 10th example, the LI widths do add up to 100% yet there's a jump at the end of the animation.
This inconsistency in behavior may indicate a bug.
Change History (14)
comment:1 Changed 11 years ago by
Component: | unfiled → css |
---|---|
Milestone: | None → 1.8 |
Owner: | set to gnarf |
Priority: | undecided → low |
Status: | new → assigned |
comment:2 Changed 11 years ago by
Resolution: | → worksforme |
---|---|
Status: | assigned → closed |
This is working on edge: http://jsfiddle.net/r5L6U/1/
All hail the great, mysterious effects rewrite!
comment:3 Changed 11 years ago by
If it only works in edge shouldn't that be fixed and not worksforme?
comment:4 Changed 11 years ago by
Resolution: | worksforme |
---|---|
Status: | closed → reopened |
I suppose. I had seen a few times where we close as WFM when we don't know what fixed it, just that it now works. I can change to just fixed.
comment:5 Changed 11 years ago by
Resolution: | → fixed |
---|---|
Status: | reopened → closed |
comment:6 Changed 11 years ago by
I'd like to try to figure out what was broken at least and write a unit for the fact that we fixed it....
comment:7 Changed 11 years ago by
feel free to reopen and write a test. I was just trying to triage open tixx. I seem to have failed :-\
comment:8 Changed 11 years ago by
Resolution: | fixed |
---|---|
Status: | closed → reopened |
Nah - it definitely closed now, but I'm going to reopen because we don't have a unit covering it, and I want to make sure we can't regress on this one, whatever magic it was that fixed it...
comment:9 Changed 11 years ago by
Status: | reopened → open |
---|
comment:10 Changed 11 years ago by
Owner: | changed from gnarf to mikesherov |
---|---|
Status: | open → assigned |
comment:11 Changed 10 years ago by
Milestone: | 1.8 → None |
---|
comment:12 Changed 10 years ago by
Milestone: | None → 1.8 |
---|---|
Owner: | changed from mikesherov to gnarf37 |
gnarf, I haven't found a good way to test this. I'm inclined to just close as fixed. If you'd like to tackle it, please do, other resolve as fixed.
comment:13 Changed 10 years ago by
Owner: | changed from gnarf37 to gnarf |
---|
comment:14 Changed 10 years ago by
Resolution: | → fixed |
---|---|
Status: | assigned → closed |
@gnarf can you take a look at this?