aboutsummaryrefslogtreecommitdiffstats
path: root/ui/widgets/accordion.js
diff options
context:
space:
mode:
authorAlexander Schmitz <arschmitz@gmail.com>2015-08-21 00:12:07 -0400
committerAlexander Schmitz <arschmitz@gmail.com>2015-08-21 08:06:24 -0400
commit009850abbfb78e2334aaf9265abf5a7705a2abf0 (patch)
tree5126b9d93d9ac8e694c10de5c49e451b1cd26487 /ui/widgets/accordion.js
parent8029cc94e123e670d2b0a80cfaa32a37435f5589 (diff)
downloadjquery-ui-009850abbfb78e2334aaf9265abf5a7705a2abf0.tar.gz
jquery-ui-009850abbfb78e2334aaf9265abf5a7705a2abf0.zip
Accordion: Style updates
Ref #14246 Ref gh-1588
Diffstat (limited to 'ui/widgets/accordion.js')
-rw-r--r--ui/widgets/accordion.js26
1 files changed, 13 insertions, 13 deletions
diff --git a/ui/widgets/accordion.js b/ui/widgets/accordion.js
index 88e9bdb6f..8c16df38f 100644
--- a/ui/widgets/accordion.js
+++ b/ui/widgets/accordion.js
@@ -53,7 +53,7 @@ return $.widget( "ui.accordion", {
header: "ui-icon-triangle-1-e"
},
- // callbacks
+ // Callbacks
activate: null,
beforeActivate: null
},
@@ -81,7 +81,7 @@ return $.widget( "ui.accordion", {
this._addClass( "ui-accordion", "ui-widget ui-helper-reset" );
this.element.attr( "role", "tablist" );
- // don't allow collapsible: false and active: false / null
+ // Don't allow collapsible: false and active: false / null
if ( !options.collapsible && ( options.active === false || options.active == null ) ) {
options.active = 0;
}
@@ -124,17 +124,17 @@ return $.widget( "ui.accordion", {
_destroy: function() {
var contents;
- // clean up main element
+ // Clean up main element
this.element.removeAttr( "role" );
- // clean up headers
+ // Clean up headers
this.headers
.removeAttr( "role aria-expanded aria-selected aria-controls tabIndex" )
.removeUniqueId();
this._destroyIcons();
- // clean up content panels
+ // Clean up content panels
contents = this.headers.next()
.css( "display", "" )
.removeAttr( "role aria-hidden aria-labelledby" )
@@ -161,7 +161,7 @@ return $.widget( "ui.accordion", {
this._super( key, value );
- // setting collapsible: false while collapsed; open first panel
+ // Setting collapsible: false while collapsed; open first panel
if ( key === "collapsible" && !value && this.options.active === false ) {
this._activate( 0 );
}
@@ -234,7 +234,7 @@ return $.widget( "ui.accordion", {
var options = this.options;
this._processPanels();
- // was collapsed or no panel
+ // Was collapsed or no panel
if ( ( options.active === false && options.collapsible === true ) || !this.headers.length ) {
options.active = false;
this.active = $();
@@ -318,7 +318,7 @@ return $.widget( "ui.accordion", {
} )
.hide();
- // make sure at least one header is in the tab order
+ // Make sure at least one header is in the tab order
if ( !this.active.length ) {
this.headers.eq( 0 ).attr( "tabIndex", 0 );
} else {
@@ -372,12 +372,12 @@ return $.widget( "ui.accordion", {
_activate: function( index ) {
var active = this._findActive( index )[ 0 ];
- // trying to activate the already active panel
+ // Trying to activate the already active panel
if ( active === this.active[ 0 ] ) {
return;
}
- // trying to collapse, simulate a click on the currently active header
+ // Trying to collapse, simulate a click on the currently active header
active = active || this.active[ 0 ];
this._eventHandler( {
@@ -436,12 +436,12 @@ return $.widget( "ui.accordion", {
options.active = collapsing ? false : this.headers.index( clicked );
- // when the call to ._toggle() comes after the class changes
+ // When the call to ._toggle() comes after the class changes
// it causes a very odd bug in IE 8 (see #6720)
this.active = clickedIsActive ? $() : clicked;
this._toggle( eventData );
- // switch classes
+ // Switch classes
// corner classes on the previously active header stay after the animation
this._removeClass( active, "ui-accordion-header-active", "ui-state-active" );
if ( options.icons ) {
@@ -467,7 +467,7 @@ return $.widget( "ui.accordion", {
var toShow = data.newPanel,
toHide = this.prevShow.length ? this.prevShow : data.oldPanel;
- // handle activating a panel during the animation for another activation
+ // Handle activating a panel during the animation for another activation
this.prevShow.add( this.prevHide ).stop( true, true );
this.prevShow = toShow;
this.prevHide = toHide;