fix(Accordion): use the hidden attribute

The edge compatibility issue is standard edge behaviour that's quickly fixed with user css, provided compat in our example
springload · Jul 2, 2020 · 8a7cdcb · 8a7cdcb
1 parent 0a1716d
commit 8a7cdcb
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/src/css/fancy-example.css b/src/css/fancy-example.css
@@ -42,7 +42,7 @@
     transform: rotate(45deg);
 }
 
-.accordion *[data-hidden] {
+[hidden] {
     display: none;
 }
 

diff --git a/src/helpers/AccordionStore.ts b/src/helpers/AccordionStore.ts
@@ -5,7 +5,7 @@ export interface InjectedPanelAttributes {
     'aria-hidden': boolean | undefined;
     'aria-labelledby': string;
     id: string;
-    'data-hidden': boolean | undefined;
+    hidden: boolean | undefined;
 }
 
 export interface InjectedHeadingAttributes {
@@ -92,7 +92,7 @@ export default class AccordionStore {
             'aria-hidden': this.allowMultipleExpanded ? !expanded : undefined,
             'aria-labelledby': this.getButtonId(uuid),
             id: this.getPanelId(uuid),
-            'data-hidden': expanded ? undefined : true,
+            hidden: expanded ? undefined : true,
         };
     };