Skip to content

script: Allow <input type=button> to be activatable#42095

Merged
mrobinson merged 1 commit intoservo:mainfrom
mrobinson:button-activatable
Jan 23, 2026
Merged

script: Allow <input type=button> to be activatable#42095
mrobinson merged 1 commit intoservo:mainfrom
mrobinson:button-activatable

Conversation

@mrobinson
Copy link
Copy Markdown
Member

@mrobinson mrobinson commented Jan 23, 2026

Although these type of inputs do not have an activation behavior (the
behavior of their clicks are up to the page), they should still get the
"active" node state when they are pressed. This allows the :active
pseudo-selector to work on them, making press highlights work.

Testing: This fixes two WPT subtests.

Although these type of inputs do not have an activation behavior (the
behavior of their clicks are up to the page), they should still get the
"active" node state when they are pressed. This allows the `:active`
pseudo-selector to work on them, making press highlights work.

Signed-off-by: Martin Robinson <[email protected]>
@mrobinson mrobinson requested a review from gterzian as a code owner January 23, 2026 07:05
@servo-highfive servo-highfive added the S-awaiting-review There is new code that needs to be reviewed. label Jan 23, 2026
@servo-highfive servo-highfive removed the S-awaiting-review There is new code that needs to be reviewed. label Jan 23, 2026
@mrobinson mrobinson added this pull request to the merge queue Jan 23, 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 Jan 23, 2026
Merged via the queue into servo:main with commit cd8df9e Jan 23, 2026
33 checks passed
@mrobinson mrobinson deleted the button-activatable branch January 23, 2026 09:00
@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 Jan 23, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants