Skip to main content

Bug Tracker

Side navigation

#1746 closed bug (wontfix)

Opened September 27, 2007 09:48PM UTC

Closed December 07, 2007 04:26AM UTC

Extra space added to table, COLSPAN attribute ignored

Reported by: Nicodemas Owned by:
Priority: minor Milestone: 1.2.2
Component: core Version: 1.2.1
Keywords: table, colspan, show, effect Cc:
Blocked by: Blocking:
Description

I ran this script in FF 2.0.0.7, Opera 9.21, and Safari for windows on a WindowsXP Pro box.

When the table rows are clicked on, the alternate row that appears has space underneath it, and it only fills one cell (it does not span both).

Please take a look :D

Attachments (2)
  • index.html (1.8 KB) - added by Nicodemas September 27, 2007 09:52PM UTC.

    Sample HTML File

  • jquery.js (45.4 KB) - added by Nicodemas September 27, 2007 09:49PM UTC.

    JQuery 1.2.1

Change History (4)

Changed September 27, 2007 09:53PM UTC by Nicodemas comment:1

The same occurs when slideDown effect is used.

Changed September 28, 2007 09:43AM UTC by Robin comment:2

Hide and show etc use display:none/block. Table rows though aren't display:block by default, they're display:table-row. So you can't expect the default methods to work on them.

Changed October 03, 2007 08:55PM UTC by Nicodemas comment:3

This is a bug, plain and simple. Using .show() without a speed parameter works just fine, but using .show("slow") or .show("fast") causes the errant behavior. You see? It's by adding the parameter the problem comes to light.

I appreciate the awesome library (I'll never go back to Prototype or Scriptaculous), but I have not the expertise to fix this. I rely on the ingenuity of great JS programmers to figure this out.

Try that, then try again at an explanation or solution. If you're reply contains anything other than "That's the way it is so STFU and deal.", I'll be very appreciative. After all, if the goal is to create a wonderful JS library loved by millions, shouldn't such errant behaviors be stamped out? I'm a programmer for corporations, and I'd be very lucky if "STFU and deal" was ever an acceptable response.

Changed December 07, 2007 04:26AM UTC by brandon comment:4

resolution: → wontfix
status: newclosed

Unfortunately in order to animate the element it needs to be display block which causes this unwanted space. As a work around try wrapping the contents of the element with a div and then animating that div.

$(...).wrapInner('<div />').show('slow');