Fixed the ui-ux changes. #233
Merged
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.
Summary
Fix node positions overlapping in Execute mode
Issue: When switching from Design to Execute mode, workflow nodes would overlap and appear in incorrect positions instead of maintaining their layout.
Solution: Fixed stale closure issue in Canvas.tsx where fitView was using outdated node positions. Changed to use reactFlowInstance.getNodes() to get current node positions instead of relying on the stale closure.
File: frontend/src/components/workflow/Canvas.tsx
Disable Rerun button while workflow is running
Issue: The "Rerun" button in the execution sidebar was enabled while the workflow was still running, which could cause confusion.
Solution: Added disabled state to Rerun buttons when the run is still in progress (live/running status). Also added tooltip explaining "Wait for run to complete".
Files:
frontend/src/components/timeline/RunSelector.tsx
frontend/src/components/timeline/ExecutionInspector.tsx
Fix scroll behavior on workflow nodes
Issue: When mouse is over a workflow component with scrollable content (error panel, text blocks, validation dock), scrolling with mouse wheel would zoom the canvas instead of scrolling the content.
Solution: Added onWheel={(e) => e.stopPropagation()} to scrollable elements to prevent wheel events from bubbling up to React Flow canvas.
Files:
frontend/src/components/workflow/ExecutionErrorView.tsx
frontend/src/components/ui/markdown.tsx
frontend/src/components/workflow/ValidationDock.tsx
Testing
bun run testbun run lintbun run typecheckDocumentation
docs/guide.md) or checked that no updates are needed..ailogs when applicable.