Bug Tracker

Modify

Ticket #1746 (closed bug: wontfix)

Opened 6 years ago

Last modified 5 years ago

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:
Blocking: Blocked by:

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

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

Change History

Changed 6 years ago by Nicodemas

JQuery 1.2.1

Changed 6 years ago by Nicodemas

Sample HTML File

comment:1 Changed 6 years ago by Nicodemas

The same occurs when slideDown effect is used.

comment:2 Changed 6 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 6 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 5 years ago by brandon

  • Status changed from new to closed
  • Resolution set to wontfix

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

Please follow the  bug reporting guidlines and use  jsFiddle when providing test cases and demonstrations instead of pasting the code in the ticket.

View

Add a comment

Modify Ticket

Action
as closed
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.