Skip to main content

Bug Tracker

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)
  • animations-bug.zip (38.4 KB) - added by hobbit June 18, 2007 11:54AM UTC.

    a test file

  • fx.diff (0.7 KB) - added by brandon June 05, 2007 02:57AM UTC.

    Patch

  • jq_bug_show.zip (19.5 KB) - added by arrix May 21, 2007 06:14AM UTC.
Change History (6)

Changed June 05, 2007 02:15AM UTC by brandon comment:1

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.

Changed June 05, 2007 02:57AM UTC by brandon 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 brandon 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 hobbit 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 john comment:5

Fixed in SVN rev [2129].

Changed June 21, 2007 02:39AM UTC by john comment:6

resolution: → fixed
status: newclosed