Skip to content

fix(mail): anchor keyboard multi-selection instead of toggling rows - #385

Open
krantheman wants to merge 1 commit into
frappe:developfrom
krantheman:fix/mail-keyboard-multiselect
Open

fix(mail): anchor keyboard multi-selection instead of toggling rows#385
krantheman wants to merge 1 commit into
frappe:developfrom
krantheman:fix/mail-keyboard-multiselect

Conversation

@krantheman

Copy link
Copy Markdown
Member

Shift+arrow toggled the row being left along with the row being entered, so walking back down a keyboard selection removed two rows per step.

The range now keeps an anchor (plus whatever was selected when it began) and recomputes anchor..cursor on every step, so it grows and shrinks by one row in either direction.

Closes #161

Shift+arrow toggled the row being left along with the row being entered, so
walking back down a selection cleared two rows per step. The range now keeps an
anchor plus the selection it started from and recomputes anchor..cursor on every
step, which shrinks and grows by one row in either direction.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@krantheman
krantheman requested a review from s-aga-r as a code owner July 29, 2026 10:04
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.

Mail: Keyboard multi-selection removes 2 items at a time when moving down

1 participant