Open
Conversation
Implement click-to-zoom animation using d3.interpolateZoom() with cubic in-out easing for natural motion transitions. Replace window resize listener with ResizeObserver to detect panel resize events, fixing circle distortion when dragging the side panel. 🤖 Assisted by ralph-orchestrator
- Reset color index at start of hidden canvas drawing for consistent mapping - Use visible canvas rect for mouse coordinates instead of hidden canvas - Fix hover and click interactions that were broken due to hidden canvas rect being zero
- Add purpose, domains, and accountabilities fields to CircleEntity - Implement Y.js synchronization for real-time circle updates - Enhanced CirclesView with detailed node information display - Improve state management between local and collaborative modes - Add node detail section showing purpose, domains, and accountabilities - Sync circles data bidirectionally between API and Y.js collaboration layer
- Add toggle buttons to switch between Plan and Circles views within PlanView component - Remove Circles as a separate top-level view from ViewSwitcher and ViewContextDefinition - Update PlanView to render CirclesView when active view is 'circles' - Maintain backward compatibility by keeping all existing functionality - Streamline navigation by consolidating related views under Plan
- Remove Card wrapper from Circles view in PlanView component - Simplify layout structure by removing unnecessary CardHeader and CardContent - Adjust CirclesView height calculation to account for reduced header space - Improve visual consistency between Plan view and Circles view layouts
…mprovements - Add currentNodeRef to prevent stale closures in callback functions - Improve node selection logic with better empty state handling - Add conditional rendering of purpose, domains, and accountabilities for roles only - Enhance organization/role node type handling with smarter default behaviors - Improve zoom behavior for root nodes (reset scale to 1 when tracking root) - Add collaboration initialization check in useCircles hook - Streamlined UI with conditional fields based on node type - Better handling of node deletion and parent-child relationships
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
1 participant
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.
Closes #54