Skip to content

feat(events): move bulk-select checkboxes inside programme event cards#255

Merged
mindsers merged 1 commit into
mainfrom
feat/programme-checkbox-in-card
Jul 16, 2026
Merged

feat(events): move bulk-select checkboxes inside programme event cards#255
mindsers merged 1 commit into
mainfrom
feat/programme-checkbox-in-card

Conversation

@mindsers

Copy link
Copy Markdown
Contributor

Summary

  • Move the bulk-select checkbox from outside each programme event card into a reserved slot inside, so cards keep uniform widths regardless of the current user's edit rights.
  • Add a selected-state card treatment (subtle tint + ring) and a stronger hover; fade the resting checkbox and reveal it on hover / check / focus.
  • Align the sticky bulk bar, per-week checkbox, and per-card checkbox on the same x-axis (pl-5 sm:pl-7 matches the card's 4px border + sm:px-6 padding).
  • Always reserve the 4px kind-color slot on the card border — cards without a kind color use transparent and stay in the same column.
  • Collapse the checkbox column entirely on weeks with zero editable events; add aria-labels to all three checkbox tiers; upgrade the sticky bar to a semantic <section>.

Test plan

  • As a Programme Manager, every card shows a faded checkbox that fills in on hover
  • Selecting a card shows the tint + ring; sticky bulk bar drops in with count and actions
  • Hovering a selected card keeps the primary tint instead of the muted hover
  • All three checkbox tiers (sticky bar / week / card) sit on the same vertical axis at both mobile (px-4) and desktop (sm:px-6) card padding
  • Cards without a kind color align with colored cards
  • Weeks where the user has no editable events collapse the checkbox column
  • Screen reader announces per-event, per-week, and bulk-action labels

The checkbox used to sit outside the card, so mixed-edit weeks produced
inconsistent card widths and the list felt broken for template
responsibles. Move the checkbox into a reserved slot inside the card,
and while we're in there:

- Add a selected-state card treatment (bg tint + ring) and stronger
  hover so bulk-select reads as a mode.
- Fade the resting checkbox and reveal on hover / check / focus.
- Align the sticky bar, week header, and card checkboxes on the same
  x-axis (pl-5 sm:pl-7).
- Always reserve the 4px kind-color slot so colorless cards align with
  colored ones.
- Collapse the checkbox column on fully-read-only weeks.
- Aria-labels on all three checkbox tiers; sticky bar upgraded to a
  semantic <section>.
@mindsers
mindsers merged commit 6913524 into main Jul 16, 2026
7 checks passed
@mindsers
mindsers deleted the feat/programme-checkbox-in-card branch July 16, 2026 08:55
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