#4038 closed bug (fixed)

Opened February 02, 2009 06:25AM UTC

Closed February 03, 2009 02:19AM UTC

sortable: cssNamespace option should be removed

Reported by:	rdworth	Owned by:
Priority:	blocker	Milestone:	1.7
Component:	ui.sortable	Version:	1.6rc6
Keywords:		Cc:
Blocked by:		Blocking:

Description

Per team discussion on

http://groups.google.com/group/jquery-ui-dev/browse_thread/thread/30b304c6407f5130

the sortable option 'cssNamespace' should be removed (not deprecated). This option was allowed for customizing the 'ui-' prefix of all UI css class names added to sortable elements. This is not compatible with the jQuery UI CSS Framework and is not intended for support by the team.

Attachments (0)

Change History (3)

Changed February 03, 2009 02:16AM UTC by rdworth comment:1

It turns out addClasses is not needed for sortable like it is for draggable and droppable.

Changed February 03, 2009 02:17AM UTC by rdworth comment:2

description:	Per team discussion on \ \ http://groups.google.com/group/jquery-ui-dev/browse_thread/thread/30b304c6407f5130 \ \ the sortable option 'cssNamespace' should be removed (not deprecated). This option had two intents \ \ 1. Allowing for customizing the 'ui-' prefix of all UI css class names added to sortable elements. This is not compatible with the jQuery UI CSS Framework and is not intended for support by the team. \ \ 2. Preventing the .addClass("ui-sortable") on many hundreds of elements at init for performance reasons, by setting cssNamespace option to false. Per the same discussion, this should instead be implemented as a new option 'addClasses' with a default value of true. \ → Per team discussion on \ \ http://groups.google.com/group/jquery-ui-dev/browse_thread/thread/30b304c6407f5130 \ \ the sortable option 'cssNamespace' should be removed (not deprecated). This option was allowed for customizing the 'ui-' prefix of all UI css class names added to sortable elements. This is not compatible with the jQuery UI CSS Framework and is not intended for support by the team. \
summary:	sortable: cssNamespace option should be removed, addClasses option added → sortable: cssNamespace option should be removed

description:

Per team discussion on \ \ http://groups.google.com/group/jquery-ui-dev/browse_thread/thread/30b304c6407f5130 \ \ the sortable option 'cssNamespace' should be removed (not deprecated). This option had two intents \ \ 1. Allowing for customizing the 'ui-' prefix of all UI css class names added to sortable elements. This is not compatible with the jQuery UI CSS Framework and is not intended for support by the team. \ \ 2. Preventing the .addClass("ui-sortable") on many hundreds of elements at init for performance reasons, by setting cssNamespace option to false. Per the same discussion, this should instead be implemented as a new option 'addClasses' with a default value of true. \ → Per team discussion on \ \ http://groups.google.com/group/jquery-ui-dev/browse_thread/thread/30b304c6407f5130 \ \ the sortable option 'cssNamespace' should be removed (not deprecated). This option was allowed for customizing the 'ui-' prefix of all UI css class names added to sortable elements. This is not compatible with the jQuery UI CSS Framework and is not intended for support by the team. \

summary:

sortable: cssNamespace option should be removed, addClasses option added → sortable: cssNamespace option should be removed

Changed February 03, 2009 02:19AM UTC by rdworth comment:3

resolution:	→ fixed
status:	new → closed

Fixed in r1972