Skip to content

Clarify entry deletion on edit page - #147

Merged
parterburn merged 2 commits into
mainfrom
paul-cursor/clarify-entry-delete-link-2b44
Aug 2, 2026
Merged

Clarify entry deletion on edit page#147
parterburn merged 2 commits into
mainfrom
paul-cursor/clarify-entry-delete-link-2b44

Conversation

@parterburn

@parterburn parterburn commented Aug 2, 2026

Copy link
Copy Markdown
Owner

Summary

  • remove the ambiguous trash icon from the entry edit header
  • add a red “Delete Entry” text link centered directly below the Update Entry button
  • retain the existing irreversible-action confirmation
  • add feature coverage for label, placement, delete method, confirmation, and icon removal

Verification

  • bundle exec rspec spec/features/entries_spec.rb — 11 examples, 0 failures
  • CI vm-job and CodeFactor pass
  • verified the final capitalization and confirmation flow at a 390×844 mobile viewport; canceling leaves the entry intact

mobile_delete_entry_label.mp4

Open in Web Open in Cursor 

cursoragent and others added 2 commits August 2, 2026 20:22
Co-authored-by: Paul Arterburn <parterburn@users.noreply.github.com>
Co-authored-by: Paul Arterburn <parterburn@users.noreply.github.com>
@parterburn
parterburn marked this pull request as ready for review August 2, 2026 21:02
@parterburn
parterburn merged commit f7da2a1 into main Aug 2, 2026
2 checks passed
@parterburn
parterburn deleted the paul-cursor/clarify-entry-delete-link-2b44 branch August 2, 2026 21:02
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