Fix: make the landing-page PDF / Offline copy links clickable - #3
Merged
Conversation
The extras row is tucked into a.talk-row's bottom padding with a negative margin, and a.talk-row is position:relative — positioned boxes paint above non-positioned siblings, so the row link covered the PDF / offline links and every click opened the talk instead. Positioning the extras row restores the correct hit-target order (later in the DOM, so it now paints on top). No visual change. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_0122f4bMszLPwGVSkuLHt3er
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
2 participants
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.
The per-talk "PDF · Offline copy" links on the landing page were unclickable — every click opened the talk deck instead of the file.
Cause: the extras row is tucked into
a.talk-row's bottom padding with a negative margin, anda.talk-rowisposition: relative. Positioned boxes paint above non-positioned siblings regardless of DOM order, so the big talk-row link covered the extras row and captured its clicks.Fix: give the extras row
position: relativetoo. It now paints (and hit-tests) on top, since it comes later in the DOM. No visual change.Verified with a click hit-test (clicks on the PDF link now resolve to the PDF link, not the talk row) and the repository audit + landing-page sync checks pass.
🤖 Generated with Claude Code
https://claude.ai/code/session_0122f4bMszLPwGVSkuLHt3er
Generated by Claude Code