Fix group mention candidate scrolling - #927
Merged
1 commit merged intoAug 24, 2026
Merged
Conversation
The mention picker stopped at eight results and keyboard navigation could move the active candidate outside the visible popup. Keep all filtered members in the bounded list and synchronize its local scroll position with the highlighted option. Constraint: Preserve existing structured mention identity and IME behavior. Rejected: Keep the eight-candidate cap | members beyond the cap would remain unreachable by scrolling. Confidence: high Scope-risk: narrow Reversibility: clean Directive: Keep keyboard highlight changes synchronized with the mention popup scroll position. Tested: Frontend npm test (89 passed); npm run build; Playwright with 12 mocked members, mouse wheel and keyboard scrolling. Not-tested: Live backend WebSocket behavior; unrelated to the local candidate picker.
yaojin3616
closed this pull request by merging all changes
into
dataelement:main
in
ae16d3e
zhangshubin01
pushed a commit
to zhangshubin01/Clawith
that referenced
this pull request
Aug 24, 2026
Bring Draft PR dataelement#927 into the local validation branch so the complete mention picker is included in the release-candidate frontend test pass. Constraint: Preserve structured mention identity and existing IME behavior. Confidence: high Scope-risk: narrow Reversibility: clean Related: dataelement#927 Tested: Merge completed without conflicts. Not-tested: Combined branch tests run after all Draft PRs are assembled.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Root cause
The mention picker capped its result set at eight members, so later members could never be reached. Arrow-key navigation also changed the highlighted index without synchronizing the popup scroll position, allowing the active option to move outside the visible area.
User impact
Users can now reach and select all matching group members with a mouse, trackpad, touch scrolling, or the arrow keys while composing an
@mention.Validation
cd frontend && npm test— 89 tests passedcd frontend && npm run build— TypeScript and Vite production build passedscrollTop=0to178scrollTop=139Not tested