Fix sidebar click#40
Conversation
Greptile SummaryThis PR refactors conversation list items in both
Confidence Score: 5/5Safe to merge — changes are purely presentational/structural and don't touch data, routing, or state management logic. All three click-target refactors follow the same pattern consistently: overlay button covers the row, interactive siblings (drag handle, dropdown trigger) are lifted with relative z-10 so they receive pointer events before the overlay. Event propagation and stacking-context behaviour are handled correctly throughout. No files require special attention. Important Files Changed
Flowchart%%{init: {'theme': 'neutral'}}%%
flowchart TD
A["Pointer Event on Row"] --> B{Which element is topmost?}
B -->|"Drag handle area\n(relative z-10)"| C["Drag handle div\nstopPropagation → DnD kit"]
B -->|"Dropdown trigger area\n(relative z-10)"| D["Dropdown trigger button\nOpens context menu"]
B -->|"Anywhere else on row\n(overlay button, no z-index)"| E["Overlay button\nabsolute inset-0"]
E --> F["onSelect / handleSelectConversation"]
F --> G["Navigate to conversation"]
Reviews (3): Last reviewed commit: "Review fixes" | Re-trigger Greptile |
No description provided.