Repository navigation
Composer | Reopen recipient list on a second To/Cc click - #11
Conversation
Co-authored-by: Cursor <cursoragent@cursor.com>
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configuration
📒 Files selected for processing (1)
Included review availability: This review used your included allowance. 4 included reviews remain after this review. Your included PR review attempts over the past 7 days set your current allowance at 5 reviews per hour. 📝 WalkthroughWalkthroughThe recipient-row Suggested reviewers: Priority: ⬇️ Low Merge Risk: ⚪ Minimal · up to The described change reopens recipient suggestions on repeat input clicks while preserving normal input behavior and excluding removal and menu interactions. No concrete merge-blocking risk is evident. 🚥 Pre-merge checks | ✅ 4✅ Passed checks (4 passed)
Comment |
Summary
After you pick someone on To or Cc, the field stays focused. A second click on that field did not reopen the people list, so you had to click away and back first.
The row mousedown handler now reopens the list when the input is already focused.
Changes
The list used to open only on
focus. Clicks on the input itself were ignored so the caret could land naturally. After a pick,focusnever fires again, so the list stayed closed.The handler still ignores chip-remove and option clicks. For a click on the already-focused input, it calls
openRecipientMenuwithoutpreventDefault, so native caret placement is unchanged.Test plan