Skip to content

Show talk and workshop descriptions inline on the schedule page#2108

Open
smagdali wants to merge 1 commit into
emfcamp:mainfrom
smagdali:line-up-inline-descriptions
Open

Show talk and workshop descriptions inline on the schedule page#2108
smagdali wants to merge 1 commit into
emfcamp:mainfrom
smagdali:line-up-inline-descriptions

Conversation

@smagdali

Copy link
Copy Markdown

The existing line-up lists every item's speaker and title but hides the description behind a click into each detail page. Better UX to render the description inline instead, so the whole programme is readable on one page. The total page size is actually still manageable (with all the descriptions added, the total page weight increases from 330k to 584k) , but much easier to explore.

Each item becomes a container div with the title as the link (descriptions contain urlized links, so the item can no longer be wrapped in a single anchor), and the description is shown with the same markup as the detail page. The favourite button stays inside the form.

I left the links to the detail page - you might argue that they're redundant now but this way you can still still share a link to a specific event.

Based on the first pass I did at https://whitelabel.org/emf-2026/ before I realised this repo was available.

Disclaimer - I made this with Claude Code.

@marksteward

marksteward commented Jun 19, 2026

Copy link
Copy Markdown
Member

Thanks for this, I think it might be a more useful fix for #2115. However, I don't think we want to make this the default - it would need to be behind a toggle button or separate "full programme" page.

You seem to have removed all visual feedback on the links, and included redundant text-decoration: underline for a link class. Were these changes to fix something affecting your browser, or just something Claude mangled?

And please remove the Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>, we don't want to be part of a move by Anthropic to prove their tools have authorship rights. I'm not sure how Anthropic's terms mesh with our licence, and it would be better to be clear about whether you're claiming ownership of these changes. If you used Claude to assist you, that's fine, but if it's vibe-coded I think it's safest to treat this as a detailed feature request instead.

@smagdali smagdali force-pushed the line-up-inline-descriptions branch from 636e67d to 7a94f1a Compare June 23, 2026 17:33
…oggle

The line-up lists each item's speaker and title but hides the description
behind a click into the detail page. Add an opt-in "Show descriptions"
toggle that reveals every description inline, so the whole programme can be
read on one page.

Descriptions are shown by default for users without JavaScript; with
JavaScript they are collapsed and the toggle (remembered per browser)
controls them, keeping the default line-up compact. Each item becomes a
container div with the title as the link, because descriptions contain links
and the item can no longer be wrapped in a single anchor; the favourite
button stays inside the form. The per-user shuffle in line_up() is unchanged.
@smagdali smagdali force-pushed the line-up-inline-descriptions branch from 7a94f1a to 2fc2f9f Compare June 26, 2026 17:22
@smagdali

Copy link
Copy Markdown
Author
  1. Descriptions are now opt-in behind a "Show descriptions" toggle on the line-up. They're collapsed by default, the choice is remembered per browser (localStorage), and the per-user shuffle in line_up() is untouched. Without JavaScript the descriptions are shown and the toggle is hidden, so nothing is lost for no-JS users.

  2. Link styling. I didn't fully understand your comment at first. I actually thought it was better to bring the link styling to be more consistent with the rest of the site, and follow best UX practice that links should be distinguishable from non-links without a mouseover. But I added the underline without the highlight colour change- it should work correctly now.

  3. Claude Authorship trailer removed. I own this, not that fancy linter. rebased to remove the original claim.

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