Skip to content

fix(TeamParticipant): keep cards side by side inside wikitables#7717

Open
Eetwalt wants to merge 1 commit into
mainfrom
tp-former-widthfix
Open

fix(TeamParticipant): keep cards side by side inside wikitables#7717
Eetwalt wants to merge 1 commit into
mainfrom
tp-former-widthfix

Conversation

@Eetwalt

@Eetwalt Eetwalt commented Jul 1, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Team participant CardsGroup uses a CSS grid with repeat(auto-fill, minmax(280px, 1fr)). Inside a shrink-to-fit wikitable cell the container width is indefinite, so auto-fill collapses to a single column and cards stacked vertically (a note-toggle reflow "fixed" it only by accident).
  • Fall back to flex-wrap with flex: 1 0 (no shrink) in that context: cards hold their min width, so the cell shrink-wraps to them and lays them side by side when they fit, wrapping when the viewport is narrow.
  • Scoped to table descendants, so the normal (non-table) grid layout is unchanged.

How did you test this change?

devtools

| image |

Copilot AI review requested due to automatic review settings July 1, 2026 11:03
@Eetwalt Eetwalt requested review from a team as code owners July 1, 2026 11:03
@Eetwalt Eetwalt added stylesheets Changes to stylesheets c: team_participant labels Jul 1, 2026

Copilot AI 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.

Pull request overview

Adjusts the TeamParticipant card group layout so cards don’t collapse into a single column when rendered inside shrink-to-fit wikitable cells, while preserving the existing grid layout outside tables.

Changes:

  • Adds a table-scoped override to render .team-participant__grid as a wrapping flex container.
  • Sets child flex sizing to preserve the minimum card width so cards stay side-by-side when space allows and wrap on narrower viewports.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants