Merge pull request #40 from github/role

muan · web-flow · commit 590e838bd880 · 2019-06-11T17:43:39.000-04:00
Avoid setting role attribute if attribute is present
diff --git a/index.js b/index.js
@@ -196,7 +196,7 @@ class DetailsDialogElement extends HTMLElement {
 
     const summary = details.querySelector('summary')
     if (summary) {
-      summary.setAttribute('role', 'button')
+      if (!summary.hasAttribute('role')) summary.setAttribute('role', 'button')
       summary.addEventListener('click', onSummaryClick, {capture: true})
     }
 

Original file line number	Diff line number	Diff line change
`@@ -196,7 +196,7 @@ class DetailsDialogElement extends HTMLElement {`
`196`	`196`
`197`	`197`	`const summary = details.querySelector('summary')`
`198`	`198`	`if (summary) {`
`199`		`- summary.setAttribute('role', 'button')`
	`199`	`+ if (!summary.hasAttribute('role')) summary.setAttribute('role', 'button')`
`200`	`200`	`summary.addEventListener('click', onSummaryClick, {capture: true})`
`201`	`201`	`}`
`202`	`202`