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

"Event: currentTarget property" is difficult to understand #28260

Closed
zjlovezj opened this issue Jul 29, 2023 · 0 comments · Fixed by #29335
Closed

"Event: currentTarget property" is difficult to understand #28260

zjlovezj opened this issue Jul 29, 2023 · 0 comments · Fixed by #29335

Comments

@zjlovezj
Copy link
Contributor

zjlovezj commented Jul 29, 2023

MDN URL

https://developer.mozilla.org/en-US/docs/Web/API/Event/currentTarget

What specific section or headline is this issue about?

description section and example section

What information was incorrect, unhelpful, or incomplete?

In description section:

It always refers to the element to which the event handler has been attached, as opposed to Event.target, which identifies the element on which the event occurred and which may be its descendant.

This is a very long sentence. And it uses too many pronouns(it/which).
It's hard to understand for readers who use English as their second language.

And the example section seems not very helpful.

What did you expect to see?

Some short sentences to describe what is a curentTarget and what is a target.

Explain what the example code designed to do and how the e.currentTarget make the job easy.

Thank you.

Do you have any supporting links, references, or citations?

No response

Do you have anything more you want to share?

A picture about event bubbling which includes target and currentTarget would be super helpful.
Maybe that's too much to ask :)

MDN metadata

Page report details
@zjlovezj zjlovezj added the needs triage Triage needed by staff and/or partners. Automatically applied when an issue is opened. label Jul 29, 2023
@github-actions github-actions bot added the Content:WebAPI Web API docs label Jul 29, 2023
@sideshowbarker sideshowbarker removed the needs triage Triage needed by staff and/or partners. Automatically applied when an issue is opened. label Jul 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants