Bug Tracker

Opened 13 years ago

Closed 12 years ago

#1746 closed bug (wontfix)

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)

jquery.js (45.4 KB) - added by Nicodemas 13 years ago.
JQuery 1.2.1
index.html (1.8 KB) - added by Nicodemas 13 years ago.
Sample HTML File

Download all attachments as: .zip

Change History (6)

Changed 13 years ago by Nicodemas

Attachment: jquery.js added

JQuery 1.2.1

Changed 13 years ago by Nicodemas

Attachment: index.html added

Sample HTML File

comment:1 Changed 13 years ago by Nicodemas

The same occurs when slideDown effect is used.

comment:2 Changed 13 years ago by Robin

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.

comment:3 Changed 13 years ago by Nicodemas

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.

comment:4 Changed 12 years ago by brandon

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');

Note: See TracTickets for help on using tickets.