Opened 5 years ago

Last modified 4 years ago

#8133 open bug

Sortable: placeholder option class gets replaced with an object

Reported by: jbruni Owned by:
Priority: minor Milestone: 2.0.0
Component: ui.sortable Version: 1.8.16
Keywords: Cc:
Blocked by: Blocking:

Description (last modified by petersendidit)

If I pass a class name as the placeholder config option when creating a sortable, I can retrieve it using sortable('option', 'placeholder') only before the placeholder is created the first time (before _createPlaceholder is called). After it, instead of returning the class name as it should, it returns an object.

See in action at http://jsfiddle.net/hyBfm/

It was easy to track the cause in the source code (jquery.ui.sortable.js)

At line 657, the option goes to a variable called "className". In the next line, 658, the option is overwritten by an object which serves internal purposes.

For us, end users, the behaviour of having inconsistent return values for the sortable('option', 'placeholder') is unwanted and it is not what the documentation says.

Change History (5)

comment:1 Changed 5 years ago by jbruni

Note: I had to "mutilate" the description until I could pass Akismet rejection.

comment:2 Changed 5 years ago by Scott González

Version: git1.8.16

comment:3 Changed 5 years ago by Scott González

Milestone: 1.9.02.0.0

comment:4 Changed 5 years ago by petersendidit

Description: modified (diff)
Status: newopen
Summary: Cannot retrieve placeholder option valueSortable: placeholder option class gets replaced with an object

comment:5 Changed 4 years ago by tj.vantoll

Related to #7121.

Note: See TracTickets for help on using tickets.