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

Fix documentation of FocusOptions.focusVisible. #34875

Merged
merged 1 commit into from
Jul 17, 2024

Commits on Jul 16, 2024

  1. Fix documentation of FocusOptions.focusVisible.

    `focusVisible: false` is not equivalent to not specifying the option as
    the current text implies.
    
    The spec is clear, and only indicates focus if focusVisible is true, or
    not specified and a browser-heuristic matches, see [1]:
    
    > If the value of the `focusVisible` dictionary member of options is
    > `true`, or is not present but in an implementation-defined way the
    > user agent determines it would be best to do so, then indicate focus.
    
    That condition doesn't match if the member is present and is `false` :)
    
    [1]: https://html.spec.whatwg.org/#dom-focus
    emilio committed Jul 16, 2024
    Configuration menu
    Copy the full SHA
    3ce398e View commit details
    Browse the repository at this point in the history