Skip to content

script: Move all focus-related code to components/script/document/focus.rs and create DocumentFocusHandler#43868

Merged
mrobinson merged 1 commit into
servo:mainfrom
mrobinson:document-focus-handler
Apr 2, 2026
Merged

script: Move all focus-related code to components/script/document/focus.rs and create DocumentFocusHandler#43868
mrobinson merged 1 commit into
servo:mainfrom
mrobinson:document-focus-handler

Conversation

@mrobinson
Copy link
Copy Markdown
Member

This continues the work to split up the large DOM structs. In this case
we create a helper struct to store focus-related state much like
DocumentEventHandler.

Testing: This is mostly code motion, so should be covered by existing tests.
Fixes: #43720

@mrobinson mrobinson requested a review from gterzian as a code owner April 2, 2026 12:12
@servo-highfive servo-highfive added the S-awaiting-review There is new code that needs to be reviewed. label Apr 2, 2026
@servo-highfive servo-highfive removed the S-awaiting-review There is new code that needs to be reviewed. label Apr 2, 2026
…cus.rs` and create `DocumentFocusHandler`

This continues the work to split up the large DOM structs. In this case
we create a helper struct to store focus-related state much like
`DocumentEventHandler`.

Signed-off-by: Martin Robinson <[email protected]>
@mrobinson mrobinson force-pushed the document-focus-handler branch from 7c98ce1 to eec09be Compare April 2, 2026 14:02
@servo-highfive servo-highfive added the S-awaiting-review There is new code that needs to be reviewed. label Apr 2, 2026
@mrobinson mrobinson enabled auto-merge April 2, 2026 14:02
@mrobinson mrobinson added this pull request to the merge queue Apr 2, 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 Apr 2, 2026
Merged via the queue into servo:main with commit ed8576b Apr 2, 2026
33 checks passed
@mrobinson mrobinson deleted the document-focus-handler branch April 2, 2026 14:49
@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 Apr 2, 2026
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.

Move focus functionality to dedicated script/dom/document/focus.rs

3 participants