Skip to content

feat: Add focusNode function to enhance node selection and view fitti…#187

Merged
felix-schultz merged 1 commit intodevfrom
feature/181-display-logs-with-associated-nodes
Aug 16, 2025
Merged

feat: Add focusNode function to enhance node selection and view fitti…#187
felix-schultz merged 1 commit intodevfrom
feature/181-display-logs-with-associated-nodes

Conversation

@felix-schultz
Copy link
Copy Markdown
Member

This pull request refactors the flow trace log viewer to improve node focusing and display of node information. The main changes include introducing a new node focusing mechanism, passing board data and a node focus callback to the Traces component, and enhancing log message rendering with node details and icons. The changes also clean up unused code and improve log level color handling for icons.

Flow node focusing and selection improvements

  • Added a new focusNode callback in FlowBoard that sets the current layer and path, then animates the view to the selected node. This replaces the previous node selection logic and ensures the correct layer is displayed before focusing the node.
  • Updated the Traces component to accept board data and an onFocusNode callback, allowing log messages to trigger node focusing in the flow board. [1] [2]

Log message rendering enhancements

  • Modified log message rendering to display node icons and friendly names, or a placeholder if the node is unknown, using the new DynamicImage and QuestionMarkCircledIcon components. [1] [2]

Code cleanup and API improvements

  • Removed unused imports and legacy node selection logic from the Traces component, simplifying the codebase. [1] [2]
  • Improved the logLevelToColor function to support color styling for icons as well as backgrounds, ensuring consistent visual feedback in the UI.wBoard

@felix-schultz felix-schultz merged commit d684f2c into dev Aug 16, 2025
2 of 4 checks passed
@felix-schultz felix-schultz deleted the feature/181-display-logs-with-associated-nodes branch August 16, 2025 17:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant