Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: the shadow root container element is ignored when clicking an element in it. #18908

Merged
merged 2 commits into from
Nov 23, 2021

Conversation

sainthkh
Copy link
Contributor

@sainthkh sainthkh commented Nov 15, 2021

User facing changelog

  • Shadow DOM elements are no longer falsely reported as hidden by their host

Additional details

  • Why was this change necessary? => When clicking an element inside a custom element, it shows the error message that it is covered by its custom element container.
  • What is affected by this change? => N/A

Any implementation details to explain?

  • I added a condition to bypass error when the element is inside a shadow root and covered by it.
  • It fixes the "covering" problem. But it doesn't solve "event messages are not sent" problem (Event listeners not working in custom elements #17531). I added a new fixture for later tests.

How has the user experience changed?

Before:

image

After:

No Error.

PR Tasks

  • Have tests been added/updated?

@cypress-bot
Copy link
Contributor

cypress-bot bot commented Nov 15, 2021

Thanks for taking the time to open a PR!

@sainthkh sainthkh marked this pull request as ready for review November 16, 2021 01:09
@sainthkh sainthkh requested a review from a team as a code owner November 16, 2021 01:09
@sainthkh sainthkh requested review from jennifer-shehane and tbiethman and removed request for a team November 16, 2021 01:09
@chrisbreiding chrisbreiding self-requested a review November 17, 2021 15:10
tbiethman
tbiethman previously approved these changes Nov 19, 2021
@chrisbreiding chrisbreiding merged commit 8c9b282 into cypress-io:develop Nov 23, 2021
tgriesser added a commit that referenced this pull request Nov 28, 2021
* develop:
  test: node_modules installs for system-tests, other improvements (#18574)
  chore(deps): update dependency semantic-release to v17.2.3 [security] (#19022)
  chore: remove flaky ci jobs for main builds (#19071)
  chore(contributing): clarify PULL_REQUEST_TEMPLATE (#19068)
  fix: the shadow root container element is ignored when clicking an element in it. (#18908)
  'Fix' flaky redirect test (#19042)
  release 9.1.0 [skip ci]
  fix: Allow 'this' to be used in overridden commands (#18899)
  fix(react): link to rerender example (#19020)
  chore(deps): update dependency aws-sdk to v2.814.0 [security] (#18948)
  fix: test config overrides leak for .only execution (#18961)
  feat: Set CYPRESS=true as env var in child processes where Cypress runs user code in Node (#18981)
  fix: Restore broken gif (#18987)
  chore: release @cypress/vite-dev-server-v2.2.1
tgriesser added a commit that referenced this pull request Nov 29, 2021
* 10.0-release:
  feat(graphql): ability to update/query for appData (#19082)
  fix system test
  fix failing tests due to merge
  resolve conflicts
  test: node_modules installs for system-tests, other improvements (#18574)
  update yarn.lock
  chore(deps): update dependency semantic-release to v17.2.3 [security] (#19022)
  chore: remove flaky ci jobs for main builds (#19071)
  chore(contributing): clarify PULL_REQUEST_TEMPLATE (#19068)
  fix: the shadow root container element is ignored when clicking an element in it. (#18908)
  'Fix' flaky redirect test (#19042)
  release 9.1.0 [skip ci]
  fix: Allow 'this' to be used in overridden commands (#18899)
  fix(react): link to rerender example (#19020)
  chore(deps): update dependency aws-sdk to v2.814.0 [security] (#18948)
  fix: test config overrides leak for .only execution (#18961)
  feat: Set CYPRESS=true as env var in child processes where Cypress runs user code in Node (#18981)
  fix: Restore broken gif (#18987)
  chore: release @cypress/vite-dev-server-v2.2.1
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Elements inside shadow dom are hidden by its host (causing them to not be interactable)
3 participants