Skip to content

Slight tweaks - #4533

Open
michaeljguarino wants to merge 1 commit into
masterfrom
final-tweaks
Open

michaeljguarino wants to merge 1 commit into
masterfrom
final-tweaks

Conversation

@michaeljguarino

Copy link
Copy Markdown
Member

mostly ui/ux tweaks, nothing big

Test Plan

Test environment: https://console.your-env.onplural.sh/

Checklist

  • I have added a meaningful title and summary to convey the impact of this PR to a user.
  • If required, I have updated the Plural documentation accordingly.
  • I have added tests to cover my changes.
  • I have deployed the agent to a test environment and verified that it works as expected (required only when changing agent code).

Plural Flow: console

mostly ui/ux tweaks, nothing big
@michaeljguarino michaeljguarino added the bug-fix This pull request fixes a bug label Oct 15, 2026
@michaeljguarino
michaeljguarino requested a review from a team as a code owner October 15, 2026 19:21
@michaeljguarino

Copy link
Copy Markdown
Member Author

plural deploy this

@soffi-ai

soffi-ai Bot commented Oct 15, 2026

Copy link
Copy Markdown
Soffi AI Summary

This PR delivers a collection of small UI/UX polish changes across the Console frontend. The most substantive changes are to the Workbench file tree panel, which gains a resizable drag handle (reusing the existing useResizablePane hook extended to support right-edge handles) and a ResizeObserver in the sidebar slot to track width changes without stale animation. Other changes clean up padding inconsistencies (removing extra paddingTop from terminal/log/raw views via ScrollablePage), reorder Workbench nav tabs so "Files" appears before "Issues" and "Alerts", center and constrain file viewer content width, remove an unwanted inset box-shadow from active SubTab components, and apply minor one-liner fixes across settings, security, and CD detail pages.

Changes

Workbench and UI polish tweaks

  • Extends useResizablePane to support both left- and right-edge drag handles with configurable storage keys, then wires a resizable drag handle onto the Workbench file tree panel. Adds a ResizeObserver in WorkbenchSidebarSlot so the sidebar animates correctly when its child resizes without a React re-render. Reorders Workbench nav tabs (Files before Issues/Alerts), centers file viewer content, removes an unwanted inset box-shadow from active SubTabs, and strips excess top padding from terminal, log, and raw-view pages. (6ea014a)

Updated: 2026-10-15 19:23 UTC

Deploy in Soffi

@github-actions

Copy link
Copy Markdown
Contributor

@greptile-apps

greptile-apps Bot commented Oct 15, 2026 •

Copy link
Copy Markdown
Contributor

RetriggerView in GreptileConfidence Score: 4/5

[Medium impact] UI layout and styling adjustments across multiple pages.

The PR appears safe to merge, with a non-blocking scrollbar interaction issue in the file sidebar.

Findings

  1. P2 Resize handle covers the scrollbar ▶

Summary

This PR adjusts page spacing and tab styling, moves Files earlier in workbench navigation, and adds a saved, resizable file sidebar.

  • You can resize the workbench file tree and keep its chosen width.
  • Workbenches show file previews in a centered reading area.
  • The workbench Files tab appears before Issues and Alerts.
  • Several console pages use more even spacing around their content.
  • Active sub-tabs no longer show an extra inset border.

Reviews (1) · Last reviewed commit: "Slight tweaks" · Reviewed by Greptile

Comment on lines +123 to +127
<DragHandleSC
tabIndex={0}
$isDragging={isDragging}
css={{ left: 'auto', right: 0 }}
{...dragHandleProps}

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

P2 Resize handle covers the scrollbar

The new DragHandleSC covers the rightmost 20 pixels of the file tree for its full height. When the file list needs scrolling, clicking or dragging its vertical scrollbar starts resizing the sidebar instead. Reserve a separate strip for the handle so users can still drag the scrollbar.

@plural-copilot plural-copilot Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Requested deployment PR created: https://github.com/pluralsh/plrl-up-demos/pull/3250

Promotes Console image tag sha-6ea014a and is annotated Plural Service: mgmt/apps.

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug-fix This pull request fixes a bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant