Skip to content

fix(editor): reduce horizontal rule click dead zone - #341

Merged
murongg merged 1 commit into
mainfrom
fix/hr-click-dead-zone
Jun 19, 2026
Merged

fix(editor): reduce horizontal rule click dead zone#341
murongg merged 1 commit into
mainfrom
fix/hr-click-dead-zone

Conversation

@murongg

@murongg murongg commented Jun 19, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Reduce horizontal rule vertical spacing in the WYSIWYG editor.
  • Limit the horizontal-rule gap guard to the small area closest to the divider so the surrounding whitespace is less likely to feel dead.
  • Update regression coverage for the narrower protected click zone and the adjusted rule spacing.

Why

#326 stopped caret jumps around horizontal rules by ignoring clicks in the rule margin. In practice that made a large area around dividers feel unresponsive. This keeps the anti-jump protection near the divider while making the surrounding whitespace less hostile to normal clicking.

Validation

  • pnpm --filter @markra/app exec vitest run src/components/MarkdownPaper.test.tsx -t "horizontal rule" passed: 4 tests.
  • pnpm --filter @markra/app exec vitest run src/styles.test.ts passed: 34 tests.
  • pnpm --filter @markra/editor build passed.
  • pnpm --filter @markra/app build passed.
  • git diff --check passed.

Risk

  • Low. The change is scoped to horizontal rule spacing and the existing horizontal-rule gap guard.
  • The issue should stay open until users can try the next build and confirm whether the interaction feels better.

Related Issues

@vercel

vercel Bot commented Jun 19, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
markra-web Ready Ready Preview, Comment Jun 19, 2026 2:43am

@murongg
murongg marked this pull request as ready for review June 19, 2026 02:46
@murongg
murongg merged commit a28f59e into main Jun 19, 2026
4 checks passed
@murongg
murongg deleted the fix/hr-click-dead-zone branch June 19, 2026 02:47
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