#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 2 years ago by tj.vantoll
- Status changed from new to open
- Summary changed from placeholder for sortable tbody elements is not visible to Sortable: placeholder for sortable tbody elements is not visible
comment:2 Changed 2 years ago by tj.vantoll
comment:3 Changed 21 months ago by Nils Heuermann
- Resolution set to fixed
- Status changed from open to 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 21 months ago by scottgonzalez
- Milestone changed from none to 1.12.0
comment:5 Changed 21 months ago by Nils Heuermann
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 21 months ago by scottgonzalez
- Milestone changed from 1.12.0 to 1.11.4
Test case: http://jsfiddle.net/tj_vantoll/sLmm00xy/