Skip to content

script: Expose the tabIndex property on HTMLOrSVGElement#42913

Merged
mrobinson merged 1 commit intoservo:mainfrom
mrobinson:tabindex
Feb 27, 2026
Merged

script: Expose the tabIndex property on HTMLOrSVGElement#42913
mrobinson merged 1 commit intoservo:mainfrom
mrobinson:tabindex

Conversation

@mrobinson
Copy link
Copy Markdown
Member

This is exposes the tabIndex property for HTMLOrSVGElement according
to the HTML specification. This is the first step toward implementation
of tab navigation in Servo.

Testing: This causes many WPT subtests to start passing, but causes a few to
start failing. This is likely because we do not have a full implementation of
the delegatesFocus parameter of attachShadow yet.
Fixes: This is part of #25001 and #32169.

@mrobinson mrobinson requested a review from gterzian as a code owner February 27, 2026 16:43
@servo-highfive servo-highfive added the S-awaiting-review There is new code that needs to be reviewed. label Feb 27, 2026
@servo-highfive servo-highfive removed the S-awaiting-review There is new code that needs to be reviewed. label Feb 27, 2026
@jdm jdm added this pull request to the merge queue Feb 27, 2026
@servo-highfive servo-highfive added the S-awaiting-merge The PR is in the process of compiling and running tests on the automated CI. label Feb 27, 2026
@mrobinson mrobinson removed this pull request from the merge queue due to a manual request Feb 27, 2026
@servo-highfive servo-highfive removed the S-awaiting-merge The PR is in the process of compiling and running tests on the automated CI. label Feb 27, 2026
This is exposes the `tabIndex` property for `HTMLOrSVGElement` according
to the HTML specification. This is the first step toward implementation
of tab navigation in Servo.

Co-authored-by: Oriol Brufau <[email protected]>
Signed-off-by: Martin Robinson <[email protected]>
@servo-highfive servo-highfive added the S-awaiting-review There is new code that needs to be reviewed. label Feb 27, 2026
@mrobinson
Copy link
Copy Markdown
Member Author

I removed and re-added this to the queue in order to properly add co-authorship with @Loirooriol.

@mrobinson mrobinson enabled auto-merge February 27, 2026 18:50
@mrobinson mrobinson added this pull request to the merge queue Feb 27, 2026
@servo-highfive servo-highfive added the S-awaiting-merge The PR is in the process of compiling and running tests on the automated CI. label Feb 27, 2026
Merged via the queue into servo:main with commit f6e44fa Feb 27, 2026
33 checks passed
@mrobinson mrobinson deleted the tabindex branch February 27, 2026 20:11
@servo-highfive servo-highfive removed the S-awaiting-merge The PR is in the process of compiling and running tests on the automated CI. label Feb 27, 2026
simonwuelker pushed a commit to simonwuelker/servo that referenced this pull request Mar 3, 2026
…2913)

This is exposes the `tabIndex` property for `HTMLOrSVGElement` according
to the HTML specification. This is the first step toward implementation
of tab navigation in Servo.

Testing: This causes many WPT subtests to start passing, but causes a
few to
start failing. This is likely because we do not have a full
implementation of
the `delegatesFocus` parameter of `attachShadow` yet.
Fixes: This is part of servo#25001 and servo#32169.

Signed-off-by: Martin Robinson <[email protected]>
Co-authored-by: Oriol Brufau <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-awaiting-review There is new code that needs to be reviewed.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants