refactor: set opened to false by default on accordion heading (#8264)

vaadin · Dec 3, 2024 · f621583 · f621583
1 parent 420d0d0
commit f621583
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 0 deletions.
diff --git a/packages/accordion/src/vaadin-accordion-heading.js b/packages/accordion/src/vaadin-accordion-heading.js
@@ -73,6 +73,7 @@ class AccordionHeading extends ActiveMixin(DirMixin(ThemableMixin(PolymerElement
       opened: {
         type: Boolean,
         reflectToAttribute: true,
+        value: false,
       },
     };
   }

diff --git a/packages/accordion/src/vaadin-lit-accordion-heading.js b/packages/accordion/src/vaadin-lit-accordion-heading.js
@@ -41,6 +41,7 @@ class AccordionHeading extends ActiveMixin(DirMixin(ThemableMixin(PolylitMixin(L
       opened: {
         type: Boolean,
         reflectToAttribute: true,
+        value: false,
       },
     };
   }

diff --git a/packages/accordion/test/dom/__snapshots__/accordion-heading.test.snap.js b/packages/accordion/test/dom/__snapshots__/accordion-heading.test.snap.js
@@ -31,6 +31,7 @@ snapshots["vaadin-accordion-heading host disabled"] =
 
 snapshots["vaadin-accordion-heading shadow default"] = 
 `<button
+  aria-expanded="false"
   id="button"
   part="content"
 >
@@ -47,6 +48,7 @@ snapshots["vaadin-accordion-heading shadow default"] =
 
 snapshots["vaadin-accordion-heading shadow disabled"] = 
 `<button
+  aria-expanded="false"
   disabled=""
   id="button"
   part="content"