Skip to content

PM-5827: Align Copilot pill with member stats - #2117

Merged
jmgasper merged 1 commit into
devfrom
PM-5827
Aug 10, 2026
Merged

PM-5827: Align Copilot pill with member stats#2117
jmgasper merged 1 commit into
devfrom
PM-5827

Conversation

@jmgasper

@jmgasper jmgasper commented Aug 9, 2026

Copy link
Copy Markdown
Collaborator

What was broken

On desktop profiles with reviewer and copilot roles, the Copilot pill began 40px to the left of the Member Stats card.

Root cause (if identifiable)

The role pills use an equal grid with a 16px gap, while TCO and Member Stats use an asymmetric flex row with a 32px gap. The independent layouts produced different second-column offsets.

What was changed

Inset only the second role card by 40px when both the desktop and two-column container thresholds are met. This preserves the shared right edge and leaves single-role and stacked layouts unchanged.

Any added/updated tests

Added a stylesheet regression assertion for the desktop and container guards plus the second-card offset.

  • yarn test:no-watch --runInBand --silent src/apps/profiles/src: passed, 16 suites and 80 tests.
  • yarn lint: passed.
  • yarn run build: passed with existing warnings.
  • yarn test:no-watch: run; 211 suites and 1,021 tests passed, while 18 unrelated suites and 32 tests failed in Engagements, Payments, Wallet Admin, and Challenge Editor areas untouched by this ticket.

What was broken

On desktop profiles with reviewer and copilot roles, the Copilot pill began 40px to the left of the Member Stats card.

Root cause (if identifiable)

The role pills use an equal grid with a 16px gap, while TCO and Member Stats use an asymmetric flex row with a 32px gap. The independent layouts produced different second-column offsets.

What was changed

Inset only the second role card by 40px when both the desktop and two-column container thresholds are met, preserving its right edge and leaving single-role and stacked layouts unchanged.

Any added/updated tests

Added a stylesheet regression assertion for the guarded desktop second-card alignment. All 80 Profiles tests, lint, and the production build pass. The full repository suite was also run; 18 unrelated suites fail in areas untouched by this ticket.
@jmgasper
jmgasper requested a review from kkartunov as a code owner August 9, 2026 23:34
@jmgasper
jmgasper merged commit 89deb1b into dev Aug 10, 2026
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant