fix(desktop): keep workspace status pill at intrinsic width - #1252
Conversation
Signed-off-by: Samuel K <skevetter@pm.me>
Signed-off-by: Samuel K <skevetter@pm.me>
✅ Deploy Preview for devsydev canceled.
|
|
Understand this PR’s impact Explore downstream dependencies and potential security impact with Blast Radius. No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Advanced Run ID: 📒 Files selected for processing (2)
Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review. 📝 WalkthroughWalkthroughThe workspace operation status container now aligns children to the cross-axis start. A test verifies the ChangesWorkspace operation layout
Priority: ⬇️ Low Estimated code review effort: 1 (Trivial) | ~5 minutes Change: Bug fix 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches✨ Simplify code
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
✅ Deploy Preview for images-devsy-sh canceled.
|
|
Tick the box to add this pull request to the merge queue (same as
|
Problem
On the Workspaces page the Status badge ("Running", "Stopped", live operation labels) stretched across the full Status column instead of rendering as a compact pill.
Cause
WorkspaceOperationwraps the badge in aflex flex-collive region. Column flex containers default toalign-items: stretch, so the badge - as a flex item - stretched to the container width, which fills the table cell.Fix
Add
items-startto the live region so the badge and the phase/error lines size to their content. One-class change; all four usages (workspaces table, workspace card, workspace detail header, create wizard progress) keep their layout.Tests
svelte-checkclean; full renderer suite green (45 files, 417 tests); full Playwright e2e suite green (55 tests) under Xvfb.Summary by CodeRabbit
Bug Fixes
Tests