Skip to content

Collapse additional preferences to two rows with a show more/less link#78

Merged
kcdragon merged 2 commits into
mainfrom
show-more-preferences
Jun 29, 2026
Merged

Collapse additional preferences to two rows with a show more/less link#78
kcdragon merged 2 commits into
mainfrom
show-more-preferences

Conversation

@kcdragon

Copy link
Copy Markdown
Collaborator

Additional-preference chips in the allocation modal could grow to many rows per category, making the modal long and noisy. Each category group now clips to two rows by default, with an inline "Show more (+N)" / "Show less" link at the end of the last visible row to reveal or re-collapse the rest. A new chip-overflow Stimulus controller measures chip positions to decide what to hide and recomputes on resize and when the dialog opens (positions aren't measurable while the dialog is display:none). Groups that already fit in two rows show no link, and a category defaults to expanded when one of its chips is already selected. Hidden chips still submit their values, so collapsing never drops a selection.

🤖 Generated with Claude Code

kcdragon and others added 2 commits June 28, 2026 21:21
Each category group of additional-preference chips now clips to two rows by
default, with an inline "Show more (+N)" / "Show less" link at the end of the
last visible row. A new chip-overflow Stimulus controller measures chip
positions to decide what to hide and recomputes on resize and when the dialog
opens. Groups that already fit in two rows show no link, and a group defaults
to expanded when one of its chips is already selected.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Resolves four CSS-parsing DoS advisories (GHSA-6jxj-px6v-747w,
GHSA-6wmf-3r64-vcwv, GHSA-8vfg-2r28-hvhj, GHSA-wwpr-jff3-395c) flagged by
bundler-audit. crass is a transitive dependency of loofah; the bump stays
within its ~> 1.0.2 constraint and touches no other gems.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@kcdragon kcdragon merged commit ab591e0 into main Jun 29, 2026
5 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