-
Notifications
You must be signed in to change notification settings - Fork 3.9k
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
Add integration test for shadow dom #20044
Comments
Here is another one: |
We a brand new issue of the same kind: |
Is this on the roadmap? cc @rudygalfi |
@rsimha let us know your thoughts here. |
I think this can be covered by E2E tests now. @prateekbh Can you comment? |
Yes, I had a conversation around the same and definitely looks like this can be done via e2e tests |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed in 7 days if no further activity occurs. Thank you for your contributions. |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed in 7 days if no further activity occurs. Thank you for your contributions. |
We've had multiple issues with DOM inconsistencies when AMP code is imported through a shadow dom based mechanism (in Polymer ), see list below. We need some shadow dom integration tests.
Here some of the issues:
#19434 amp-experiment
#19694 amp-subscriptions-google
#12912 amp-ads
#12388 amp-img
#12151 amp-list
#10799 amp-access
The text was updated successfully, but these errors were encountered: