Opened 16 years ago
Closed 15 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)
Change History (6)
Changed 16 years ago by
comment:2 Changed 16 years ago by
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 15 years ago by
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 15 years ago by
Resolution: | → wontfix |
---|---|
Status: | new → closed |
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');
JQuery 1.2.1