Opened 3 years ago

Closed 3 years ago

Last modified 3 years ago

#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 3 years ago by tj.vantoll

Status: newopen
Summary: placeholder for sortable tbody elements is not visibleSortable: placeholder for sortable tbody elements is not visible

comment:3 Changed 3 years ago by Nils Heuermann

Resolution: fixed
Status: openclosed

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 3 years ago by Scott González

Milestone: none1.12.0

comment:5 Changed 3 years 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 3 years ago by Scott González

Milestone: 1.12.01.11.4
Note: See TracTickets for help on using tickets.