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

Include "input" event to the list of events in the Element documentation page. #37493

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

mohammednumaan
Copy link
Contributor

Description

As per the documentation, input event belongs to the Element, however, the Element page in the documentation does not list this event. This PR fixes the inconsistency.

  • Added "Input" event to the list of events in the Element documentation page.

Motivation

It helps ensure consistency across the documentation and avoids confusion to readers.

Additional details

Additional details can be found in the issue linked below.

Related issues and pull requests

Fixes #37481

@mohammednumaan mohammednumaan requested a review from a team as a code owner January 4, 2025 08:08
@mohammednumaan mohammednumaan requested review from wbamberg and removed request for a team January 4, 2025 08:08
@github-actions github-actions bot added Content:WebAPI Web API docs size/xs [PR only] 0-5 LoC changed labels Jan 4, 2025
Copy link
Contributor

github-actions bot commented Jan 4, 2025

This pull request has merge conflicts that must be resolved before it can be merged.

@github-actions github-actions bot added the merge conflicts 🚧 [PR only] label Jan 4, 2025
@github-actions github-actions bot removed the merge conflicts 🚧 [PR only] label Jan 4, 2025
Copy link
Contributor

github-actions bot commented Jan 4, 2025

Preview URLs

Flaws (1)

URL: /en-US/docs/Web/API/Element
Title: Element
Flaw count: 1

  • macros:
    • /en-US/docs/Web/API/Element/getBoxQuads does not exist

(comment last updated: 2025-01-06 12:02:45)

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.

Thank you! This list should be alphabetical, so input should be listed after contentvisibilityautostatechange.

@Josh-Cena
Copy link
Member

@mohammednumaan Are you going to make the suggested change?

@mohammednumaan
Copy link
Contributor Author

@Josh-Cena yeah. I'll make the change.

@mohammednumaan
Copy link
Contributor Author

@Josh-Cena I've made the change.

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/xs [PR only] 0-5 LoC changed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Missing "input" event
3 participants