Fix UI bugs: logs, textarea overflow, schedule names#297
Open
krishna9358 wants to merge 4 commits intomainfrom
Open
Fix UI bugs: logs, textarea overflow, schedule names#297krishna9358 wants to merge 4 commits intomainfrom
krishna9358 wants to merge 4 commits intomainfrom
Conversation
… glitches - Add always-visible 'click to expand' hint for truncated logs (ExecutionInspector.tsx) - Cap textarea height to 160px with internal scrolling to prevent input fields from blocking screen (ParameterField.tsx) - Truncate long schedule names with ellipsis and tooltip; wrap action buttons to prevent off-screen layout breaks (ConfigPanel.tsx, WorkflowSchedulesPanel.tsx) Signed-off-by: Krishna Mohan <krishanmohank974@gmail.com>
|
Codex usage limits have been reached for code reviews. Please check with the admins of this repo to increase the limits by adding credits. |
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> Signed-off-by: Krishna Mohan <krishanmohank974@gmail.com>
|
Codex usage limits have been reached for code reviews. Please check with the admins of this repo to increase the limits by adding credits. |
Move the "click to expand" hint from inline (only visible after horizontal scroll) to a sticky-positioned overlay that stays pinned at the right edge of the visible log panel on hover. Signed-off-by: Krishna Mohan <krishanmohank974@gmail.com>
Replace native title attribute with Radix Tooltip on schedule names in ConfigPanel and WorkflowSchedulesPanel so users see the full name in a styled popover on hover. Signed-off-by: Krishna Mohan <krishanmohank974@gmail.com>
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.
Summary
Fixes three UI bugs affecting the workflow builder experience:
1. No way to see truncated logs
Issue: In the execution logs panel, truncated log messages (>150 chars) only showed a hidden "⋯" indicator that required hovering to discover. Users had no way to know logs were truncated or that clicking would reveal the full content.
Solution: Replaced the invisible hover-only indicator with an always-visible "click to expand" hint in a subtle blue color (
text-blue-400/70), making the interaction discoverable.File:
frontend/src/components/timeline/ExecutionInspector.tsx2. Too many inputs block the screen
Issue: Textarea parameter fields (
textareaandjsontypes) had no height constraint. Components with multiple textarea parameters (e.g., Prowler Security Scan with 4+ text fields) caused the config panel to be dominated by oversized input fields, blocking the rest of the UI.Solution: Added
max-h-[160px] overflow-y-autoto textarea fields and capped initial rows to max 4 viaMath.min(). Users can still manually resize via the drag handle, but initial render no longer fills the screen.File:
frontend/src/components/workflow/ParameterField.tsx3. UI glitch in schedules — entry point
Issue: Schedule names in the entry point config panel had no truncation. Very long schedule names pushed action buttons (Pause, Run now, Edit, Delete) off-screen and broke the card layout. Same issue existed in the schedules sidebar.
Solution: Added
truncate min-w-0to schedule name spans withtitletooltip for full name on hover. Addedflex-shrink-0to status badges. Changed ConfigPanel schedule card to verticalflex-collayout withflex-wrapon action buttons to prevent off-screen overflow. Applied same truncation fix inWorkflowSchedulesSidebar.Files:
frontend/src/components/workflow/ConfigPanel.tsx,frontend/src/components/workflow/WorkflowSchedulesPanel.tsxTesting
bun run test— 648 pass, 118 skip, 0 failDocumentation