-
Notifications
You must be signed in to change notification settings - Fork 22.5k
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
Update FocusEvent pages #14127
Update FocusEvent pages #14127
Conversation
Preview URLs (7 pages)Flaws (7)Note! 6 documents with no flaws that don't need to be listed. 🎉 URL:
External URLs (2)URL: URL:
(this comment was updated 2022-09-28 00:17:32.865423) |
@danburzo from what I can see the changes suggested have been made - can you confirm? If you resolve conflicts we should be able to get this pr through 👍 |
* Note that focusin/focusout follow their non-bubbling counterparts; * Note that focusin/focusout don't (yet) have an event handling property; * Add the description of the FocusEvent.relatedTarget property on all events; * Link to HTMLElement.focus() / .blur() methods from the similarly-named event pages.
@Rumyra I've adapted my proposed changes to the new structure for FocusEvent pages. Substantial new things:
|
@jpmedley Is this ok to merge? |
Summary
I've made the following updates:
Fixes #14084.
Motivation
I propose these changes to make the FocusEvent pages more useful to readers by matching what they will experience in browsers.
Supporting details
onfocusin
/onfocusout
IDL attributes be added? whatwg/html#4542Related issues
Fixes #14084.
Metadata