Repository navigation
Fix tree lines, column alignment, and tooltips in the Relationships tab - #4065
Conversation
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
| <Glyph class={showExpandIcon ? 'shrink-0' : 'invisible shrink-0'} /> | ||
| </div> | ||
|
|
||
| <style lang="postcss"> |
There was a problem hiding this comment.
The fallback for browsers without container query units can probably go. Every browser we support already handles cqw and container-type, so the indent prop and the inline calc(18% + ${indent * 0.18}px) variables above, as well as the extra @screen blocks here never take effect.
The indent={generation * 16} math is also tied to the pl-4 on the children wrapper. If that padding changes, the fallback breaks without anyone noticing.
Without the fallback, the whole <style> block can become Tailwind classes:
- ID column: lg:w-[18cqw]
- Timestamps column: lg:w-[28cqw] xl:w-[42cqw] lg:grid-cols-[minmax(0,1fr)_minmax(0,1fr)_8rem]
That also removes the display: grid that repeats the lg:grid class.
There was a problem hiding this comment.
Thank you. Just updated and tested again. UI changes still look as intended.
Every supported browser handles cqw and container-type, so the indent-based calc() fallback never applied. Use Tailwind classes for the column widths and data-testid hooks in the spec. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Description & motivation 💭
Designer-led polish for the Relationships tab in workflow details. It builds on the Parent view branch lines in #4064, but doesn't depend on that PR.
border-secondary, matching the table branch lines in Align child workflow bolts and add branch lines in Parent view #4064.cqw, withcontainer-type: inline-sizeon the list). The first column absorbs each row's indentation. Before, nested rows were narrower, so every column started at a different x, and rows with a +/− icon shifted further.rounded-lg border border-primaryas the paginated tables.overflow-cliprounds the inner corners without breaking the graph's sticky positioning.ZoomSvgis only used by this tab.Screenshots (if applicable) 📸
Dark mode, 1440px wide, Expand All on,
shippingrow active, hovering Zoom in.main)Design Considerations 🎨
18cqwID,28cqw/42cqwtimes atlg/xl,8remduration) were tuned so Start/End fit on two lines and durations on one at 1440px. At narrowlgwidths (~1100px) timestamps still wrap to several lines, as they do onmain.Testing 🧪
How was this tested 👻
Checked against a local server with nested, failing, running and fan-out child workflows, in light and dark mode at 1440px and 1100px. Measured in the browser that every row's columns start at the same x, that column labels share the same top, that ticks stop short of badges, and that the graph still sticks under the top nav when scrolling.
Steps for others to test: 🚶🏽♂️🚶🏽♀️
🤖 Generated with Claude Code