Droppable : disabled should not have ui-state-disabled

[wojes]

When disabling droppable element through its "disable" method, it gets ui-state-disabled. It is not the case though when initializing the droppable through disabled option set to true. This class sets the opacity of the element which is not desirable as droppable is an interaction (which should not affect the styling) not a widget

[JamesKhoury]

Reproducible paste bin: ​http://jsfiddle.net/MByPK/1/

Shows it is the same with Draggable.

Also shows an issue where the opacity of the .ui-state-disabled, .ui-widget-content .ui-state-disabled, .ui-widget-header .ui-state-disabled rule displays the div infront of the draggable's background (chrome & ff5/win 7). (If you disable the rule via the developer tools it shows in the correct place. Probably a browser bug.)

Comment in ui.button.js:

TODO: pull out $.Widget's handling for the disabled option into $.Widget.prototype._setOptionDisabled so it's easy to proxy and can be overridden by individual plugins this._setOption( "disabled", options.disabled );

Would suggest this is an issue with all the widgets? Although maybe the display of the disabled widget is upto the css?

[mikesherov]

related to #5974 and #5973

[Mike Sherov]

Widget: Stop setting ui-state-disabled and aria by default on setting disabled option. Fixes #5973 - Resizable: disabled should not have the ui-state-disabled class or aria attribute aria-disabled Fixes #5974 - Draggable: disabled should not have the ui-state-disabled class or aria attribute aria-disabled Fixes #6039 - Droppable : disabled should not have ui-state-disabled

Changeset: 4d67f4f34908a50ef88f95df4103ee001b777e45

[Scott González]

Reverted to keep this out of a patch release.

[Scott González]

Widget: Stop setting ui-state-disabled and aria by default on setting disabled option. Fixes #5973 - Resizable: disabled should not have the ui-state-disabled class or aria attribute aria-disabled Fixes #5974 - Draggable: disabled should not have the ui-state-disabled class or aria attribute aria-disabled Fixes #6039 - Droppable: disabled should not have ui-state-disabled

This reverts commit 23771d38ba9d2663f6db0243c8e992dc7ff6844a.

Changeset: 44d07173db32b498e5f83f60db290ff1463daee3