Skip to content

fix: access ShadowRoot via window.ShadowRoot#9050

Merged
christian-bromann merged 1 commit intowebdriverio:v7from
klipstein:v7
Oct 19, 2022
Merged

fix: access ShadowRoot via window.ShadowRoot#9050
christian-bromann merged 1 commit intowebdriverio:v7from
klipstein:v7

Conversation

@klipstein
Copy link
Contributor

Pull for v7 branch of #9046.

* fix: access ShadowRoot via window.ShadowRoot

fixes an issue in browsers where ShadowRoot is not available globally (e.g. Edge 18). Error message is "ShadowRoot is not defined".

A similar issue was reported here vuejs/test-utils#293 and the fix was referencing ShadowRoot from window instead: https://github.com/vuejs/core/pull/2943/files

* Revert accidentally changed return value
Copy link
Member

@christian-bromann christian-bromann left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@christian-bromann christian-bromann added the PR: Bug Fix 🐛 PRs that contain bug fixes label Oct 19, 2022
@christian-bromann christian-bromann merged commit b6658e5 into webdriverio:v7 Oct 19, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

PR: Bug Fix 🐛 PRs that contain bug fixes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants