Ticket #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: | |
| 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
Change History
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.


JQuery 1.2.1