Skip to content

fix: keep line after deleting empty bullet#27

Open
dkarter wants to merge 1 commit into
mainfrom
fix-empty-bullet-keeps-line
Open

fix: keep line after deleting empty bullet#27
dkarter wants to merge 1 commit into
mainfrom
fix-empty-bullet-keeps-line

Conversation

@dkarter

@dkarter dkarter commented Jun 17, 2026

Copy link
Copy Markdown
Member

Motivation

Pressing Return on an empty bullet with delete_last_bullet_if_empty enabled removed the whole line, moving the cursor up. The spec now requires the bullet marker to be cleared while keeping the cursor on that line.

Summary of Changes

  • Route whitespace-only bullet text through the shared empty-bullet handling.
  • Remove the special-case insert-mode deletion path that used ddi.
  • Update LC-003 coverage and spec text to assert that the blank line remains.

Testing

  • mise run test:spec LC-003
  • mise run test:spec LC-004
  • mise run test:spec LC-005
  • mise run openspec:check
  • mise run test
  • mise run fmt:check

Dependencies/Special Considerations

none

@dkarter dkarter marked this pull request as ready for review June 17, 2026 05:40
@dkarter dkarter added this pull request to the merge queue Jun 17, 2026
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to no response for status checks Jun 17, 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.

1 participant