Merge pull request #465 from warrenspe/collapsible-default

wuda-io · web-flow · commit 34aeaf3b4a75 · 2024-04-03T10:50:03.000+02:00
fix: class="active" non-accordion collapsible not opening by default
diff --git a/src/collapsible.ts b/src/collapsible.ts
@@ -68,12 +68,12 @@ export class Collapsible extends Component<CollapsibleOptions> {
 
     // Open active
     const activeBodies: HTMLElement[] = Array.from(this.el.querySelectorAll('li.active > .collapsible-body'));
-    if (this.options.accordion)
+    if (this.options.accordion) {
       if (activeBodies.length > 0) {
         // Accordion => open first active only
         this._setExpanded(activeBodies[0]);
       }
-    else {
+    } else {
       // Expandables => all active
       activeBodies.forEach(el => this._setExpanded(el));
     }

Original file line number	Diff line number	Diff line change
`@@ -68,12 +68,12 @@ export class Collapsible extends Component<CollapsibleOptions> {`
`68`	`68`
`69`	`69`	`// Open active`
`70`	`70`	`const activeBodies: HTMLElement[] = Array.from(this.el.querySelectorAll('li.active > .collapsible-body'));`
`71`		`- if (this.options.accordion)`
	`71`	`+ if (this.options.accordion) {`
`72`	`72`	`if (activeBodies.length > 0) {`
`73`	`73`	`// Accordion => open first active only`
`74`	`74`	`this._setExpanded(activeBodies[0]);`
`75`	`75`	`}`
`76`		`- else {`
	`76`	`+ } else {`
`77`	`77`	`// Expandables => all active`
`78`	`78`	`activeBodies.forEach(el => this._setExpanded(el));`
`79`	`79`	`}`