From 902b08cc0f6022d76f772e0b50a2996091c864b0 Mon Sep 17 00:00:00 2001
From: Cory LaViska <cory@abeautifulsite.net>
Date: Thu, 13 Apr 2023 12:47:48 -0400
Subject: [PATCH] revert role and don't use <header> for buttons

---
 src/components/details/details.ts | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/src/components/details/details.ts b/src/components/details/details.ts
index 6ed06fb25..6c9f9094c 100644
--- a/src/components/details/details.ts
+++ b/src/components/details/details.ts
@@ -170,10 +170,11 @@ export default class SlDetails extends ShoelaceElement {
           'details--rtl': isRtl
         })}
       >
-        <header
+        <div
           part="header"
           id="header"
           class="details__header"
+          role="button"
           aria-expanded=${this.open ? 'true' : 'false'}
           aria-controls="content"
           aria-disabled=${this.disabled ? 'true' : 'false'}
@@ -191,7 +192,7 @@ export default class SlDetails extends ShoelaceElement {
               <sl-icon library="system" name=${isRtl ? 'chevron-left' : 'chevron-right'}></sl-icon>
             </slot>
           </span>
-        </header>
+        </div>
 
         <div class="details__body">
           <slot part="content" id="content" class="details__content" role="region" aria-labelledby="header"></slot>