Rebuild Amsterdam 2026 PQC agenda for a denser, deeper programme#687
Open
vanbroup wants to merge 4 commits into
Open
Rebuild Amsterdam 2026 PQC agenda for a denser, deeper programme#687vanbroup wants to merge 4 commits into
vanbroup wants to merge 4 commits into
Conversation
Redesign the three-day grid around 30-min sessions, 45-min panels, one 60-min "deep" panel each morning, a 90-min lunch, and uniform 30-min breaks, netting more programming slots per room per day without losing depth. Breakout rooms A-E now run two waves a day plus a longer single-room roundtable block. Wednesday trials asynchronous hall formats: the red hall runs a 60-min panel while the blue hall runs a 4x15-min lightning round in the same window. To support sessions spanning multiple grid rows without the old "continuation" filler-card convention, sessions can now declare their own durationMinutes; the agenda renderer computes a rowSpan and merges the card across rows via explicit CSS Grid placement instead. Also adds Sven Rajala (PQC Working Group Vice Chair) to the speaker list and rotates conference open/close hosting between the three chairs, rewrites the draft-agenda notice, and fixes a pre-existing callout-warning CSS bug (display:flex on the paragraph itself split inline bold text into separate wrapping flex items).
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
Contributor
There was a problem hiding this comment.
Code Review
This pull request updates the conference agenda layout and content, notably introducing support for sessions to declare custom durations and span multiple rows in the agenda grid. It also adds a new speaker, updates session timings, and switches the callout warning style from flexbox to a floated layout to prevent text wrapping issues. The review feedback highlights two critical template safety issues: a potential nil pointer dereference if $locationsData is missing, and a build failure risk due to Go templates' non-short-circuiting and function when evaluating missing .durationMinutes values.
Important
The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.
Deploying with
|
| Status | Name | Latest Commit | Preview URL | Updated (UTC) |
|---|---|---|---|---|
| ✅ Deployment successful! View logs |
pkic-org | 22599a1 | Commit Preview URL Branch Preview URL |
Jul 11 2026, 01:47 PM |
Pull the 20 currently-accepted proposals from the PKI Consortium CFP system into the agenda, with real abstracts, speaker bios, and headshots, respecting per-speaker confirmation status and committee placement notes (breakout-only sessions, strategic vs technical hall fit, Day 3 availability constraints). Rebuild the agenda grid's row-sizing as a deterministic calendar: each row's height is a function of real elapsed minutes plus a per-row content floor (title length, speaker count, abstract presence) distributed correctly across multi-row sessions, so parallel sessions of different durations render proportionally without forcing unrelated cards to stretch, overlap, or clip their titles/speakers/duration badge. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
The per-row content floor accounted for title/speakers/abstract but not the optional Watch/Slides card-footer (.youtube/.presentation), so sessions with those links overflowed their row and visually bled into the break row below. Sponsored breaks with a logo also needed more room than a plain "Break" label — the fixed 92px guess still left the card taller than its row; recalibrated to 114px against the actual rendered height so it gets the same breathing room as every other break. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Address Gemini Code Assist review on PR #687: guard $locationsData before dereferencing .order (nil if an event lacks locations in front matter), and split the durationMinutes and/gt check into nested ifs since Go templates evaluate all args of `and` eagerly, which would error on `add` with a nil durationMinutes. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Redesign the three-day grid around 30-min sessions, 45-min panels, one 60-min "deep" panel each morning, a 90-min lunch, and uniform 30-min breaks, netting more programming slots per room per day without losing depth. Breakout rooms A-E now run two waves a day plus a longer single-room roundtable block. Wednesday trials asynchronous hall formats: the red hall runs a 60-min panel while the blue hall runs a 4x15-min lightning round in the same window.
To support sessions spanning multiple grid rows without the old "continuation" filler-card convention, sessions can now declare their own durationMinutes; the agenda renderer computes a rowSpan and merges the card across rows via explicit CSS Grid placement instead.
Also adds Sven Rajala (PQC Working Group Vice Chair) to the speaker list and rotates conference open/close hosting between the three chairs, rewrites the draft-agenda notice, and fixes a pre-existing callout-warning CSS bug (display:flex on the paragraph itself split inline bold text into separate wrapping flex items).