Skip to content

Conversation

@mdroidian
Copy link
Contributor

@mdroidian mdroidian commented Aug 25, 2025

https://www.loom.com/share/e033ad05e72a46a6b92a19761257fa2c

  • Implemented drag-and-drop capabilities for nodes and relations within the DiscourseGraphPanel.
  • Utilized useMemo for performance optimization in panel item generation and unique relations.
  • Introduced drag state management using useAtom for better control over drag events.
  • Refactored event handlers for pointer interactions to improve user experience.
  • Updated UI to reflect dragging state with a preview of the dragged item.

@linear
Copy link

linear bot commented Aug 25, 2025

@supabase
Copy link

supabase bot commented Aug 25, 2025

This pull request has been ignored for the connected project zytfjzqyijgagqxrzbmz because there are no changes detected in packages/database/supabase directory. You can change this behaviour in Project Integrations Settings ↗︎.


Preview Branches by Supabase.
Learn more about Supabase Branching ↗︎.

Copy link
Contributor Author

@mdroidian mdroidian left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@CodeRabbit Full Review

…e management

- Implemented drag-and-drop capabilities for nodes and relations within the DiscourseGraphPanel.
- Utilized useMemo for performance optimization in panel item generation and unique relations.
- Introduced drag state management using useAtom for better control over drag events.
- Refactored event handlers for pointer interactions to improve user experience.
- Updated UI to reflect dragging state with a preview of the dragged item.
@mdroidian mdroidian merged commit f800d6c into main Aug 25, 2025
5 checks passed
@mdroidian mdroidian deleted the eng-773-implement-dragdrop-nodes branch August 25, 2025 01:25
@github-project-automation github-project-automation bot moved this to Done in General Aug 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

No open projects
Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants