#10682 closed bug (fixed)
Sortable: placeholder for sortable tbody elements is not visible
Reported by: | oemmes | Owned by: | |
---|---|---|---|
Priority: | minor | Milestone: | 1.11.4 |
Component: | ui.sortable | Version: | 1.11.2 |
Keywords: | Cc: | ||
Blocked by: | Blocking: |
Description
When using a sortable for tbody elements of a table the created placeholder (when dragging an tbody element) is not visible.
The reason is that the placeholder is an empty tbody. The tbody needs a tr and td which gives it a height to be visible. This is already implemented for tr placeholders which also need a td element in it.
I created a pull request on github to fix this: https://github.com/jquery/jquery-ui/pull/1380
Change History (6)
comment:1 Changed 9 years ago by
Status: | new → open |
---|---|
Summary: | placeholder for sortable tbody elements is not visible → Sortable: placeholder for sortable tbody elements is not visible |
comment:2 Changed 9 years ago by
comment:3 Changed 9 years ago by
Resolution: | → fixed |
---|---|
Status: | open → closed |
Sortable: Append a tr with td to the placeholder of tbody elements
When sorting tbody elements of a table the placeholder needs to have a tr with td elements to be visible. The appended elements are created in the same way as for the placeholder of a tr element; the first row of the sorted tbody is used for that.
Fixes #10682 Closes gh-1380
Changeset: 962e05dc1d0a51a7458bc44725417aa3462cd89a
comment:4 Changed 9 years ago by
Milestone: | none → 1.12.0 |
---|
comment:5 Changed 9 years ago by
Sortable: Append a tr with td to the placeholder of tbody elements
When sorting tbody elements of a table the placeholder needs to have a tr with td elements to be visible. The appended elements are created in the same way as for the placeholder of a tr element; the first row of the sorted tbody is used for that.
Fixes #10682 Closes gh-1380 (cherry picked from commit 962e05dc1d0a51a7458bc44725417aa3462cd89a)
Changeset: ddc1b32a45746db7cc8e96de92818936464b323b
comment:6 Changed 9 years ago by
Milestone: | 1.12.0 → 1.11.4 |
---|
Test case: http://jsfiddle.net/tj_vantoll/sLmm00xy/