Skip to main content

Search and Top Navigation

#10682 closed bug (fixed)

Opened November 03, 2014 08:17AM UTC

Closed February 24, 2015 02:37PM UTC

Last modified March 10, 2015 04:34PM UTC

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

Attachments (0)
Change History (6)

Changed November 03, 2014 02:24PM UTC by tj.vantoll comment:1

status: newopen
summary: placeholder for sortable tbody elements is not visibleSortable: placeholder for sortable tbody elements is not visible

Changed November 03, 2014 02:27PM UTC by tj.vantoll comment:2

Changed February 24, 2015 02:37PM UTC by Nils Heuermann comment:3

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

Changed February 24, 2015 02:37PM UTC by scottgonzalez comment:4

milestone: none1.12.0

Changed March 10, 2015 04:31PM UTC by Nils Heuermann comment:5

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

Changed March 10, 2015 04:34PM UTC by scottgonzalez comment:6

milestone: 1.12.01.11.4