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

Commit a123d54

Browse files
committed
Collapsible: Add support for same expanded/collapsed non default icon
Fixes #4801 Collapsible Content: Info Icon Issue. Having the same non standard icon for expanded and collapsed state, toggleClass removes the ui-icon-x class for collapsed state. Tests: Passed.
1 parent 8610bad commit a123d54

File tree

1 file changed

+3
-2
lines changed

1 file changed

+3
-2
lines changed

js/widgets/collapsible.js

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -122,13 +122,14 @@ $.widget( "mobile.collapsible", $.mobile.widget, {
122122
event.preventDefault();
123123

124124
collapsibleHeading
125-
.toggleClass( "ui-collapsible-heading-collapsed", isCollapse)
125+
.toggleClass( "ui-collapsible-heading-collapsed", isCollapse )
126126
.find( ".ui-collapsible-heading-status" )
127127
.text( isCollapse ? o.expandCueText : o.collapseCueText )
128128
.end()
129129
.find( ".ui-icon" )
130130
.toggleClass( "ui-icon-" + expandedIcon, !isCollapse )
131-
.toggleClass( "ui-icon-" + collapsedIcon, isCollapse )
131+
// logic or cause same icon for expanded/collapsed state would remove the ui-icon-class
132+
.toggleClass( "ui-icon-" + collapsedIcon, ( isCollapse || expandedIcon == collapsedIcon ) )
132133
.end()
133134
.find( "a" ).first().removeClass( $.mobile.activeBtnClass );
134135

0 commit comments

Comments
 (0)