Skip to content

Sit org sectors & age groups side by side - #2246

Merged
maebeale merged 1 commit into
mainfrom
maebeale/sectors-age-columns
Aug 17, 2026
Merged

Sit org sectors & age groups side by side#2246
maebeale merged 1 commit into
mainfrom
maebeale/sectors-age-columns

Conversation

@maebeale

Copy link
Copy Markdown
Collaborator

🤖 suggested review level: 1 Skim 👀 view-only ERB/Tailwind layout change on one page

Goal

  • On the org show page, Sectors and Age groups served should sit next to each other as columns, wrapping on narrow viewports.

Change

  • They were separate cells in the 5-col info grid, so Affiliations (3) + Sectors (2) filled the first row and Age groups wrapped under Affiliations.
  • Grouped both into one col-span-2 cell laid out flex flex-col sm:flex-row — two columns on desktop, stacked when narrow.
  • Matches the person show page pattern.
  • Kept the 3-sector cap, ... overflow, "Explore sector data" link, and both feature-flag guards.

🤖 Generated with Claude Code

They were separate cells in the 5-col info grid, so Affiliations + Sectors
filled the first row and Age groups wrapped underneath Affiliations. Group
them into one col-span-2 cell laid out as two columns that stack on narrow
viewports, matching the person show page.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Copilot AI lite review requested due to automatic review settings August 17, 2026 13:15

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.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@maebeale
maebeale marked this pull request as ready for review August 17, 2026 13:17
@maebeale
maebeale merged commit 014da8a into main Aug 17, 2026
3 checks passed
@maebeale
maebeale deleted the maebeale/sectors-age-columns branch August 17, 2026 13:27
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.

2 participants