Skip to content

Clarify editor:kill-line description - #41

Open
ryukiyokoo1164 wants to merge 1 commit into
inkdropapp:mainfrom
ryukiyokoo1164:ryukiyokoo1164-patch-1
Open

Clarify editor:kill-line description#41
ryukiyokoo1164 wants to merge 1 commit into
inkdropapp:mainfrom
ryukiyokoo1164:ryukiyokoo1164-patch-1

Conversation

@ryukiyokoo1164

Copy link
Copy Markdown

The description for editor:kill-line currently reads "Deletes the line at the cursor," which can be misread as deleting the entire line.

As confirmed by @craftzdog on the forum (https://forum.inkdrop.app/t/sublime-plugin-doesnt-work-on-v6/5577), editor:kill-line actually works the same way as Emacs' traditional kill-line: it deletes from the cursor to the end of the line, not the whole line.

This PR updates the description to make that behavior clearer.

The description for `editor:kill-line` currently reads "Deletes the line at the cursor," which can be misread as deleting the entire line.

As confirmed by @craftzdog on the forum (https://forum.inkdrop.app/t/sublime-plugin-doesnt-work-on-v6/5577), `editor:kill-line` actually works the same way as Emacs' traditional `kill-line`: it deletes from the cursor to the end of the line, not the whole line.

This PR updates the description to make that behavior clearer.
@vercel

vercel Bot commented Aug 19, 2026

Copy link
Copy Markdown

@ryukiyokoo1164 is attempting to deploy a commit to the inkdrop Team on Vercel.

A member of the Team first needs to authorize it.

### editor:kill-line {{ selector: '.cm-editor' }}

Deletes the line at the cursor.
Deletes from the cursor to the end of the line, in the style of Emacs' `kill-line`.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you drop the "in the style of Emacs' kill-line" part?

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.

2 participants