-
Notifications
You must be signed in to change notification settings - Fork 5
feat: add DetailPanel and extend UserHoverCard to all tables #133
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
DetailPanel Component: - Create slide-in DetailPanel component for issue details - Implement smooth slide-in/out animations with backdrop - Add escape key and backdrop click to close - Prevent body scroll when panel is open - Display comprehensive issue information: - Title, ID, repository - Author with UserHoverCard integration - Created/updated dates - Priority badge with color coding - Labels with color styling - Comments count - Full description with line breaks - Additional info (language, stars, type) - Responsive design (full-width on mobile, 600px on desktop) State Management: - Create detailPanel store with Zustand - openPanel, closePanel, setIssue actions - Centralized state for selectedIssue and isOpen UI Integration: - Add UserHoverCard to action-required table author column - Add DetailPanel to action-required page with row click - Add DetailPanel to quick-wins table with row click - Prevent panel opening when clicking buttons/links/inputs - Apply hover effects to clickable rows All tables now display user hover cards with scores and clickable rows open the DetailPanel for detailed information.
…-01REMSaoZ3VK6uVAFhJsa6vM feat: add DetailPanel and extend UserHoverCard to all tables
|
Caution Review failedThe pull request is closed. WalkthroughA detail panel feature is introduced using a new DetailPanel component and Zustand store to manage panel state. Integration points in the action-required and quick-wins pages enable clickable rows that open the detail panel for issue inspection. Changes
Sequence Diagram(s)sequenceDiagram
participant User
participant Page as Page/Component
participant Store as useDetailPanelStore
participant Panel as DetailPanel
User->>Page: Click table row
Page->>Store: openPanel(issue)
Store->>Store: Set selectedIssue + isOpen=true
Store-->>Panel: Trigger re-render
Panel->>Panel: Render with issue data
Panel-->>User: Display slide-over panel
User->>Panel: Press Escape or click backdrop
Panel->>Store: onClose()
Store->>Store: Set isOpen=false
Store-->>Panel: Trigger re-render
Panel-->>User: Hide panel
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes
Possibly related PRs
Poem
✨ Finishing touches
🧪 Generate unit tests (beta)
📜 Recent review detailsConfiguration used: CodeRabbit UI Review profile: CHILL Plan: Pro 📒 Files selected for processing (5)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
DetailPanel Component:
State Management:
UI Integration:
All tables now display user hover cards with scores and
clickable rows open the DetailPanel for detailed information.
Summary by CodeRabbit