Merge pull request #57 from github/root

muan · web-flow · commit 5cad25281edb · 2020-06-10T11:34:38.000-04:00
Fix getRootNode not available in Edge 18
diff --git a/src/index.ts b/src/index.ts
@@ -100,7 +100,7 @@ function toggle(event: Event): void {
   if (!(dialog instanceof DetailsDialogElement)) return
 
   if (details.hasAttribute('open')) {
-    const root = dialog.getRootNode() as Document | ShadowRoot
+    const root = 'getRootNode' in dialog ? (dialog.getRootNode() as Document | ShadowRoot) : document
     if (root.activeElement instanceof HTMLElement) {
       initialized.set(dialog, {details, activeElement: root.activeElement})
     }

Original file line number	Diff line number	Diff line change
`@@ -100,7 +100,7 @@ function toggle(event: Event): void {`
`100`	`100`	`if (!(dialog instanceof DetailsDialogElement)) return`
`101`	`101`
`102`	`102`	`if (details.hasAttribute('open')) {`
`103`		`- const root = dialog.getRootNode() as Document \| ShadowRoot`
	`103`	`+ const root = 'getRootNode' in dialog ? (dialog.getRootNode() as Document \| ShadowRoot) : document`
`104`	`104`	`if (root.activeElement instanceof HTMLElement) {`
`105`	`105`	`initialized.set(dialog, {details, activeElement: root.activeElement})`
`106`	`106`	`}`