Opened 2 years ago

Closed 2 years ago

Last modified 2 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 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:3 Changed 2 years 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 2 years ago by scottgonzalez

  • Milestone changed from none to 1.12.0

comment:5 Changed 2 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 2 years ago by scottgonzalez

  • Milestone changed from 1.12.0 to 1.11.4
Note: See TracTickets for help on using tickets.