Skip to content

New pages: ShadowRoot.elementFromPoint() #37115

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

Merged
merged 16 commits into from
Jan 29, 2025
Merged

New pages: ShadowRoot.elementFromPoint() #37115

merged 16 commits into from
Jan 29, 2025

Conversation

estelle
Copy link
Member

@estelle estelle commented Dec 6, 2024

@estelle estelle requested a review from a team as a code owner December 6, 2024 02:43
@estelle estelle requested review from wbamberg and removed request for a team December 6, 2024 02:43
@github-actions github-actions bot added Content:WebAPI Web API docs size/m [PR only] 51-500 LoC changed labels Dec 6, 2024
Copy link
Contributor

github-actions bot commented Dec 6, 2024

Preview URLs

(comment last updated: 2025-01-29 19:29:57)

Comment on lines 36 to 37
let customElem = document.querySelector("my-custom-element");
let shadow = customElem.shadowRoot;
Copy link
Collaborator

Choose a reason for hiding this comment

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

Could be const.

@estelle estelle requested a review from wbamberg January 12, 2025 03:00
@wbamberg
Copy link
Collaborator

@estelle , since you have some comments from Hamish here I'll wait for them to be resolved before re-reviewing (otherwise things might get confusing).

estelle and others added 2 commits January 23, 2025 15:54
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@estelle estelle requested a review from hamishwillee January 23, 2025 23:54
An {{domxref("Element")}}; the topmost shadow root element located at the specified coordinates, if any.

## Examples

Copy link
Collaborator

@hamishwillee hamishwillee Jan 24, 2025

Choose a reason for hiding this comment

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

FYI it is now common, but not mandatory (AFAIK) to always have a descriptive level 3 heading as well as the main heading. This is a good thing to do, but up to you.

Copy link
Member Author

Choose a reason for hiding this comment

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

the example is just a code example, not a rendered example. I generally add the subheading on live examples or when there are multiple. here is would be overkill.

Copy link
Collaborator

@hamishwillee hamishwillee left a comment

Choose a reason for hiding this comment

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

@wbamberg My main issue has now been addressed by @estelle - you can review around my remaining comments if you like.

@estelle
Copy link
Member Author

estelle commented Jan 29, 2025

@wbamberg I think this is ready for re-review

Copy link
Collaborator

@wbamberg wbamberg left a comment

Choose a reason for hiding this comment

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

👍 thanks Estelle!

@wbamberg wbamberg merged commit a359b3b into main Jan 29, 2025
12 checks passed
@wbamberg wbamberg deleted the shadowroot branch January 29, 2025 19:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Content:WebAPI Web API docs size/m [PR only] 51-500 LoC changed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants