Skip to content
This repository was archived by the owner on Oct 8, 2021. It is now read-only.

Commit 07c3339

Browse files
author
Gabriel Schulhof
committed
[collapsibleSet] Update first/last classes using common behaviour
1 parent 4c9cb3d commit 07c3339

File tree

1 file changed

+6
-12
lines changed

1 file changed

+6
-12
lines changed

js/widgets/collapsibleSet.js

Lines changed: 6 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@
55
//>>css.structure: ../css/structure/jquery.mobile.collapsible.css
66
//>>css.theme: ../css/themes/default/jquery.mobile.theme.css
77

8-
define( [ "jquery", "../jquery.mobile.widget", "./collapsible" ], function( $ ) {
8+
define( [ "jquery", "../jquery.mobile.widget", "./collapsible", "./addFirstLastClasses" ], function( $ ) {
99
//>>excludeEnd("jqmBuildExclude");
1010
(function( $, undefined ) {
1111

@@ -64,26 +64,20 @@ $.widget( "mobile.collapsibleset", $.mobile.widget, {
6464
},
6565

6666
_refresh: function( create ) {
67-
var $el = this.element,
68-
o = this.options,
69-
collapsiblesInSet = $el.children( ":jqmData(role='collapsible')" );
67+
var collapsiblesInSet = this.element.children( ":jqmData(role='collapsible')" );
7068

7169
$.mobile.collapsible.prototype.enhance( collapsiblesInSet.not( ".ui-collapsible" ) );
72-
collapsiblesInSet.removeClass( "ui-first-child ui-last-child" );
73-
if ( !create ) {
74-
collapsiblesInSet = collapsiblesInSet.filter( ":visible" );
75-
}
76-
collapsiblesInSet
77-
.eq( 0 ).addClass( "ui-first-child" )
78-
.end()
79-
.last().addClass( "ui-last-child" );
70+
71+
this._addFirstLastClasses( collapsiblesInSet, this._getVisibles( collapsiblesInSet, create ), create );
8072
},
8173

8274
refresh: function() {
8375
this._refresh( false );
8476
}
8577
});
8678

79+
$.widget( "mobile.collapsibleset", $.mobile.collapsibleset, $.mobile.behaviors.addFirstLastClasses );
80+
8781
//auto self-init widgets
8882
$( document ).bind( "pagecreate create", function( e ) {
8983
$.mobile.collapsibleset.prototype.enhanceWithin( e.target );

0 commit comments

Comments
 (0)