UN-3741 [FIX] Keep list-row owner and updated columns aligned across rows#2193
Open
athul-rs wants to merge 2 commits into
Open
UN-3741 [FIX] Keep list-row owner and updated columns aligned across rows#2193athul-rs wants to merge 2 commits into
athul-rs wants to merge 2 commits into
Conversation
The meta cluster is anchored on the row's right edge, so content-sized columns shifted "Owned By" sideways row by row depending on the length of the owner label. Pin the owner and updated columns; bold the title to match the agentic Prompt Studio project list. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
The meta wrapper was still shrinkable, so once its columns became fixed they overflowed the box and overlapped the action icons (26px at 1440, 66px at 1024). Pin the wrapper too and let the left column, which ellipsizes, absorb the shortfall. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Contributor
Frontend Lint Report (Biome)✅ All checks passed! No linting or formatting issues found. |
Contributor
WalkthroughList view metadata columns now preserve fixed sizing during resizing, while adapter titles use a semibold font weight. ChangesList View Styling
Estimated code review effort: 2 (Simple) | ~10 minutes Suggested reviewers: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
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 |
|
Contributor
|
| Filename | Overview |
|---|---|
| frontend/src/components/widgets/list-view/ListView.css | Aligns metadata columns with fixed flex bases and strengthens list-title typography; current shared ListView variants retain sufficient space and existing ellipsis behavior. |
Reviews (1): Last reviewed commit: "UN-3741 [FIX] Stop the pinned meta colum..." | Re-trigger Greptile
Contributor
Unstract test resultsPer-group results
Critical paths❌ Regressions (must be zero)
|
athul-rs
requested review from
chandrasekharan-zipstack,
jaseemjaskp and
ritwik-g
July 22, 2026 08:56
jaseemjaskp
approved these changes
Jul 22, 2026
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
2 participants
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.
What
Keeps the
Owned ByandUpdatedcolumns of the shared list rows at the samex position on every row, and bolds the list title to match the agentic Prompt
Studio project list.
Why
QA on staging (rc.384) reported two issues on the adapter and Prompt Studio
list pages:
(1) is a regression from #2182. That PR replaced the row's fixed-width columns
(
.list-view-content{width:80%}+.adapters-list-title-container{width:50%})with a flex layout. The meta cluster is anchored on the row's right edge — the
left column grows to absorb the row's slack — so a content-sized owner column
starts wherever its own text ends up: x=985 for a long email, x=1207 for
Me.(2) is not a regression:
.adapters-list-titlehas never carried afont-weight, while the agentic list sets600on its own title class. The twolists only look inconsistent when compared side by side.
How
Three CSS rules in the shared
ListView:.adapters-list-profile-container→flex: 0 0 240px(avatar + label + the200px-capped email), left-aligned so the label starts at a fixed x
.list-view-modified-container→flex: 0 0 170px(fits the longest string,Updated a few seconds ago).list-view-meta→flex: 0 0 autoso the wrapper cannot shrink below itsnow-rigid children and spill them over the action icons. The left column
absorbs the shortfall instead — it ellipsizes, this doesn't.
Widths are pinned per sub-column rather than on the wrapper, so pages that show
the owner but no
Updated(Connectors, Workflows) reserve no dead space.Can this PR break any existing features. If yes, please list possible items. If no, please explain why. (PS: Admins do not merge the PR without this section filled)
CSS-only, scoped to the four pages that use the shared
ListView(adapters viaTool Settings, Prompt Studio, Connectors, Workflows). No JSX, no props, no API.
The risk worth naming: the two fixed column widths are magic numbers. If a
future column is added to the meta cluster, or the owner label stops being
capped at 200px, they need revisiting. Verified on a deployed build that the
row never overflows, the action icons are never clipped or overlapped, and the
title/owner fall back to their existing ellipsis + tooltip.
Database Migrations
None.
Env Config
None.
Relevant Docs
None.
Related Issues or PRs
Follow-up to #2182.
Dependencies Versions
None.
Notes on Testing
Verified against a deployed build in a dev namespace, measuring live geometry
rather than eyeballing. Rows were stress-varied (short
Me, long email,Me +1co-owner, 50-char service account, mixed relative times, mixed/absent
descriptions).
x-position spread across rows (0 = perfectly aligned):
Owned ByUpdatedUpdatedright edge vs actions left edgeWith the fix reverted in-page, the owner spread returns to 181-222px, which is
the reported bug.
Also checked: no row overflow or clipped action icons down to 1024; the left
column degrades 645 -> 188px through the existing title ellipsis; owner-only
pages collapse the meta cluster to exactly 240px; over-long owner labels
ellipsize at the 200px cap with the full value still in the tooltip.
Not covered: no automated test — this is layout CSS and the repo has no visual
regression harness. Connectors and Workflows were exercised by removing the
Updatedblock from a live page rather than with real rows, since the test orghas none.
Screenshots
Checklist
I have read and understood the Contribution Guidelines.