Side navigation
#1214 closed bug (fixed)
Opened May 21, 2007 06:07AM UTC
Closed June 21, 2007 02:39AM UTC
.show(...) doesn't always make elements visible
Reported by: | arrix | Owned by: | |
---|---|---|---|
Priority: | major | Milestone: | 1.1.3 |
Component: | effects | Version: | 1.1.2 |
Keywords: | Cc: | ||
Blocked by: | Blocking: |
Description
1.1.3a in IE/FF/Opera
.show(...) doesn't always make elements visible.
e.g.
$('#div1').slideDown().hide().show('slow'); $('#div2').fadeIn().hide().show('slow'); $('#div3').hide('slow').show('slow');
Please test with attached file. The elements end up with style="display:none"
Attachments (3)
Change History (6)
Changed June 05, 2007 02:15AM UTC by comment:1
Changed June 05, 2007 02:57AM UTC by comment:2
Replying to [comment:1 brandon]:
I've attached a patch for this. The logic was reversed causing the animate method to bail out at the wrong time. However, this patch seems to break the fx test suite which I have yet to look into.
I just got totally confused on the logic. The new patch seemingly fixes the problems.
Changed June 05, 2007 03:11AM UTC by comment:3
Replying to [comment:2 brandon]:
Replying to [comment:1 brandon]: > I've attached a patch for this. The logic was reversed causing the animate method to bail out at the wrong time. However, this patch seems to break the fx test suite which I have yet to look into. I just got totally confused on the logic. The new patch seemingly fixes the problems.
hmmm... I'm just completely off target on this one. The current attached patch doesn't work for all test cases. If I comment out lines 311-315 in fx.js all test cases work just fine. It seems the enhancement ticket #1051 has caused more harm than good. Another good test-case for this bug can be found in #1234.
BTW... I'm pretty sure this is the bug that is causing the Tabs plugin to fail with 1.1.3a.
Changed June 18, 2007 12:00PM UTC by comment:4
I added a test file to the attachments:
<html> <head> <title>Test file</title> <style> #item { display: none; } </style> <script type="text/javascript" src="jquery-1.1.3a.js"></script> <script type="text/javascript"> $(document).ready(function(){ $("#item").hide(1); }); </script> </head> <body> <a href="#" onclick="$('#item').show(1);">This doesn't work: show(1)</a><br> <a href="#" onclick="$('#item').show(0);">This works: show(0)</a><br><br> <a href="#" onclick="$('#item').hide(1);">This doesn't work: hide(1)</a><br> <a href="#" onclick="$('#item').hide(0);">This works: hide(0)</a><br><br> <div id="item">The animated item</div> </body></html>
Changed June 21, 2007 02:39AM UTC by comment:5
Fixed in SVN rev [2129].
Changed June 21, 2007 02:39AM UTC by comment:6
resolution: | → fixed |
---|---|
status: | new → closed |
I've attached a patch for this. The logic was reversed causing the animate method to bail out at the wrong time. However, this patch seems to break the fx test suite which I have yet to look into.