Skip to content

script: Implement insertLineBreak command - #47445

Merged
TimvdLippe merged 3 commits into
servo:mainfrom
Psychpsyo:execCommand-insertLineBreak
Aug 23, 2026
Merged

TimvdLippe merged 3 commits into
servo:mainfrom
Psychpsyo:execCommand-insertLineBreak

Conversation

@Psychpsyo

Copy link
Copy Markdown
Contributor

Implement the insertLineBreak command for contenteditable stuff.

Testing: More WPT passes

PS: Yes, I do still plan to get to writing is_collapsed_line_break at some point.
But for now, I want at least something implemented for every editing command. (at least all the ones that are relevant to keyboard input so that once regular DOM content becomes selectable, we can put in a thing that calls them upon user key presses. :) )

Signed-off-by: Psychpsyo (Cameron) <60073468+Psychpsyo@users.noreply.github.com>
@Psychpsyo
Psychpsyo requested a review from gterzian as a code owner August 22, 2026 08:49
@servo-highfive servo-highfive added the S-awaiting-review There is new code that needs to be reviewed. label Aug 22, 2026
Comment thread tests/wpt/meta/editing/run/insertlinebreak.html.ini Outdated
Comment thread components/script/dom/execcommand/commands/insertlinebreak.rs
Comment thread components/script/dom/execcommand/commands/insertlinebreak.rs
Signed-off-by: Psychpsyo (Cameron) <60073468+Psychpsyo@users.noreply.github.com>
Signed-off-by: Psychpsyo (Cameron) <60073468+Psychpsyo@users.noreply.github.com>
@servo-highfive servo-highfive removed the S-awaiting-review There is new code that needs to be reviewed. label Aug 23, 2026
@TimvdLippe
TimvdLippe added this pull request to the merge queue Aug 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 Aug 23, 2026
Merged via the queue into servo:main with commit f9b4d1e Aug 23, 2026
33 checks passed
@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 Aug 23, 2026
sagudev added a commit to sagudev/servo that referenced this pull request Aug 24, 2026
{"fail_fast": false, "matrix": [{"name": "Linux (Production, WPT)", "workflow": "linux", "wpt": true, "profile": "production", "unit_tests": false, "devtools_tests": false, "build_libservo": false, "bencher": false, "coverage": false, "capi": false, "build_args": "", "wpt_args": "_webgpu --processes 1", "number_of_wpt_chunks": 20}]}
RichardTjokroutomo pushed a commit to RichardTjokroutomo/servo that referenced this pull request Oct 6, 2026
Implement the `insertLineBreak` command for contenteditable stuff.

Testing: More WPT passes

PS: Yes, I do still plan to get to writing `is_collapsed_line_break` at
some point.
But for now, I want at least something implemented for every editing
command. (at least all the ones that are relevant to keyboard input so
that once regular DOM content becomes selectable, we can put in a thing
that calls them upon user key presses. :) )

---------

Signed-off-by: Psychpsyo (Cameron) <60073468+Psychpsyo@users.noreply.github.com>
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