Skip to content

Focus note editor blank space#5872

Open
ComputelessComputer wants to merge 1 commit into
fix/remove-insights-tabfrom
fix/note-editor-blank-space-focus
Open

Focus note editor blank space#5872
ComputelessComputer wants to merge 1 commit into
fix/remove-insights-tabfrom
fix/note-editor-blank-space-focus

Conversation

@ComputelessComputer

@ComputelessComputer ComputelessComputer commented Jul 2, 2026

Copy link
Copy Markdown
Collaborator

Add a shared trailing body-line command and wire blank note-surface clicks to it.


Note

Low Risk
Localized note-editing UX and focus/selection behavior with new unit tests; no auth, persistence, or API contract changes beyond the editor command surface.

Overview
Clicking empty padding around the session note editor now moves the caret to the trailing body line (reusing or inserting an empty paragraph) instead of a generic focus.

The shared ProseMirror helper focusTrailingEmptyLine is exported and exposed as focusAtTrailingEmptyLine on note editor commands. NoteInput switches the scroll container from onClick to onMouseDown, ignores non–left-clicks and clicks inside .ProseMirror, and routes the rest to that command. The in-editor trailing-empty-line plugin still uses the same helper, including title-only notes that need a new body paragraph.

Reviewed by Cursor Bugbot for commit 62c2562. Bugbot is set up for automated code reviews on this repo. Configure here.

@ComputelessComputer ComputelessComputer force-pushed the fix/remove-insights-tab branch from fbec601 to 5a4e0bd Compare July 2, 2026 05:26
@ComputelessComputer ComputelessComputer force-pushed the fix/note-editor-blank-space-focus branch from f956235 to f96dc27 Compare July 2, 2026 05:26
Add a shared trailing body-line command and wire blank note-surface clicks to it.
@ComputelessComputer ComputelessComputer force-pushed the fix/remove-insights-tab branch from 5a4e0bd to 3d553d6 Compare July 2, 2026 08:34
@ComputelessComputer ComputelessComputer force-pushed the fix/note-editor-blank-space-focus branch from f96dc27 to 62c2562 Compare July 2, 2026 08:34
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