Skip to content

fix(schedule): don't split poster collapse on start time (regression from #289)#292

Merged
JacobCoffee merged 1 commit intomainfrom
fix-poster-collapse-regression
May 1, 2026
Merged

fix(schedule): don't split poster collapse on start time (regression from #289)#292
JacobCoffee merged 1 commit intomainfrom
fix-poster-collapse-regression

Conversation

@JacobCoffee
Copy link
Copy Markdown
Member

Summary

PR #289 added `slot.start` to the collapsedGroups key to stop morning+afternoon Break slots merging into one 10:30-4:30 row. Side effect: every 5-minute `kind="poster"` slot got its own key too, so the daily poster session exploded into 50+ rows on the Sunday schedule.

Only append start to the key for `kind="break"`. Posters keep the original `kind-day-name` key and collapse to one row again.

Test plan

  • Sun schedule: poster session shows as ONE "Posters" row at the start time, not 50 individual 5-minute rows.
  • Sat / Wed posters (if any): same.
  • Friday's morning Break and afternoon Break still render as TWO separate rows (the fix(schedule): don't collapse same-named breaks across the day #289 fix stays intact for breaks).
  • Per-room same-start lunches still merge into one row.

🤖 Generated with Claude Code

…sion from #289)

PR #289 added \`-\${slot.start}\` to the collapsedGroups key to stop
morning + afternoon Break slots from merging into one 10:30-4:30 row.
Side effect: every individual 5-minute kind="poster" slot got its own
key too, so the daily poster session — which is supposed to fold into
ONE "Posters" row per day — exploded into 50+ 5-minute rows on the
schedule timeline.

Only append the start segment for kind="break". Posters keep the
original \`kind-day-name\` key and collapse to a single row again.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@JacobCoffee JacobCoffee merged commit 71394e3 into main May 1, 2026
2 checks passed
@JacobCoffee JacobCoffee deleted the fix-poster-collapse-regression branch May 1, 2026 15:28
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