mirror of
https://github.com/jquery/jquery-ui.git
synced 2025-01-07 20:34:24 +00:00
Accordion: Removed deprecated icon option. Fixes #6835 - Accordion: Remove icons.headerSelected.
This commit is contained in:
parent
09eed560f8
commit
fd952c5a79
@ -9,9 +9,8 @@ TestHelpers.commonWidgetTests( "accordion", {
|
||||
header: "> li > :first-child,> :not(li):even",
|
||||
heightStyle: "auto",
|
||||
icons: {
|
||||
"activeHeader": null,
|
||||
"header": "ui-icon-triangle-1-e",
|
||||
"headerSelected": "ui-icon-triangle-1-s"
|
||||
"activeHeader": "ui-icon-triangle-1-s",
|
||||
"header": "ui-icon-triangle-1-e"
|
||||
},
|
||||
|
||||
// callbacks
|
||||
|
@ -104,22 +104,6 @@ test( "{ active: jQuery Object }", function() {
|
||||
|
||||
|
||||
|
||||
module( "accordion (deprecated) - icons", setupTeardown() );
|
||||
|
||||
test( "icons, headerSelected", function() {
|
||||
expect( 3 );
|
||||
var element = $( "#list1" ).accordion({
|
||||
icons: { headerSelected: "a1", header: "h1" }
|
||||
});
|
||||
ok( element.find( ".ui-accordion-header.ui-state-active span.ui-icon" ).hasClass( "a1" ) );
|
||||
element.accordion( "option", "icons", { headerSelected: "a2", header: "h2" } );
|
||||
ok( !element.find( ".ui-accordion-header.ui-state-active span.ui-icon" ).hasClass( "a1" ) );
|
||||
ok( element.find( ".ui-accordion-header.ui-state-active span.ui-icon" ).hasClass( "a2" ) );
|
||||
});
|
||||
|
||||
|
||||
|
||||
|
||||
|
||||
module( "accordion (deprecated) - resize", setupTeardown() );
|
||||
|
||||
|
17
ui/jquery.ui.accordion.js
vendored
17
ui/jquery.ui.accordion.js
vendored
@ -544,23 +544,6 @@ $.widget( "ui.accordion", {
|
||||
|
||||
// DEPRECATED
|
||||
if ( $.uiBackCompat !== false ) {
|
||||
// icon options
|
||||
(function( $, prototype ) {
|
||||
$.extend( prototype.options.icons, {
|
||||
activeHeader: null, // remove default so we fall back to old values
|
||||
headerSelected: "ui-icon-triangle-1-s"
|
||||
});
|
||||
|
||||
var _createIcons = prototype._createIcons;
|
||||
prototype._createIcons = function() {
|
||||
if ( this.options.icons ) {
|
||||
this.options.icons.activeHeader = this.options.icons.activeHeader ||
|
||||
this.options.icons.headerSelected;
|
||||
}
|
||||
_createIcons.call( this );
|
||||
};
|
||||
}( jQuery, jQuery.ui.accordion.prototype ) );
|
||||
|
||||
// expanded active option, activate method
|
||||
(function( $, prototype ) {
|
||||
prototype.activate = prototype._activate;
|
||||
|
Loading…
Reference in New Issue
Block a user