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 references should list bubbles, cancelable, and composed values #19590

Open
lapcat opened this issue Aug 15, 2022 · 11 comments
Open

Event references should list bubbles, cancelable, and composed values #19590

lapcat opened this issue Aug 15, 2022 · 11 comments
Labels
Content:WebAPI Web API docs needs decision The task needs consensus through discussion

Comments

@lapcat
Copy link

lapcat commented Aug 15, 2022

MDN URL

https://developer.mozilla.org/en-US/docs/Web/API/Element/click_event

What specific section or headline is this issue about?

No response

What information was incorrect, unhelpful, or incomplete?

The event reference documentation did not give the values of the bubbles, cancelable, and composed properties.

What did you expect to see?

The values of the bubbles, cancelable, and composed properties of the event.

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

The old documentation used to give the values of the bubbles and cancelable properties:

https://web.archive.org/web/20220104000330/https://developer.mozilla.org/en-US/docs/Web/API/Element/click_event

Do you have anything more you want to share?

I gave an example of one event, "click", but this issue applies to every event reference.

@github-actions github-actions bot added Content:WebAPI Web API docs needs triage Triage needed by staff and/or partners. Automatically applied when an issue is opened. labels Aug 15, 2022
@sideshowbarker sideshowbarker removed the needs triage Triage needed by staff and/or partners. Automatically applied when an issue is opened. label Aug 15, 2022
@hamishwillee
Copy link
Collaborator

@queengooborg The tables showing the bubbles, cancellable, etc were removed as part of your PR here: #18226

What was the thinking?

@teoli2003
Copy link
Contributor

Vinyl used the new event template for this. So she is not guilty there.

When we defined the new template, we noticed that most of the time, it was always the same values, so we decided to use prose to mark when an event was bubbling or cancelable.

I would be open to revisiting it (not reverting but finding a 3rd solution), but I would like to understand the use cases better. Note that if it bubbles, it is written.

(Overall, I'm not happy with how the bubbling chain is documented – and this was also the case when we had the table. So maybe both problems should be thought together, as a whole)

@hamishwillee
Copy link
Collaborator

@teoli2003 (and @queengooborg) Thanks for letting us know. Can you own this discussion - I'm not back until Friday?

I don't have much to add other than pulling information about of prose is not always easy, and I didn't assume that because there was no comment (in the case I looked at) that it meant the event did not bubble; I assumed we'd dropped the information or omitted it. In other words, having the prose works when you know the pattern.

@teoli2003
Copy link
Contributor

We should launch a discussion on mdn/discussion. I'm a bit under the water now, but I will try to formulate something to launch it. If you don't see any activity from me by Monday, please ping me (nicely).

@hamishwillee
Copy link
Collaborator

@teoli2003 OK! Typically when I need to kick people off for more work, I like to yell "work harder, not smarter". I hope this will work on you :-)

@hamishwillee
Copy link
Collaborator

@teoli2003 ^^^ When you have a moment.

@T1mL3arn
Copy link

After some time googling "does %eventname% ibubbes" and don't understanding why such info is not on MDN I tried to look on webarchive and bingo - the info was finally found. Wanted to create a similar issue and I am not the first one.

@teoli2003

we noticed that most of the time, it was always the same values so we decided to use prose to mark when an event was bubbling or cancelable.

Didn't find the "prose" for change event, for click event, and I guess for many other events it is the same.

Note that if it bubbles, it is written.

At this moment - it isn't. Maybe I am missing something and there are some general info on other pages about it. For now it looks like such info is removed completely.

but I would like to understand the use cases better

The use case was simple - immediately get desired info:

  • open event doc
  • instantly see summary about bubbling/cancelable/etc

how it is now:

  • open event doc
  • see no summary about bubbling/cancelable/etc
  • Ctrl+F and see nothing again
  • go google or webarchive

gbj pushed a commit to leptos-rs/leptos that referenced this issue Mar 4, 2023
gbj pushed a commit to leptos-rs/leptos that referenced this issue Mar 21, 2023
@wbamberg
Copy link
Collaborator

There's more detail on this issue at #31137 (comment).

@hamishwillee
Copy link
Collaborator

As per that linked issues

But we're not going around this loop again ... until I run into the next case I have to update.

@teoli2003 I think you're going to have to accept that the current approach does not work.

@wbamberg
Copy link
Collaborator

wbamberg commented Apr 1, 2024

See https://github.com/orgs/mdn/discussions/665 - let's try to get a resolution for this.

@teoli2003
Copy link
Contributor

As per that linked issues

But we're not going around this loop again ... until I run into the next case I have to update.

@teoli2003 I think you're going to have to accept that the current approach does not work.

Sounds good to me. The current approach indeed doesn't work.

@Josh-Cena Josh-Cena added the needs decision The task needs consensus through discussion label Jun 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Content:WebAPI Web API docs needs decision The task needs consensus through discussion
Projects
None yet
Development

No branches or pull requests

7 participants