Skip to content

Add photography section with masonry grid and lightbox#8

Open
csherland wants to merge 12 commits into
masterfrom
add-photography-section
Open

Add photography section with masonry grid and lightbox#8
csherland wants to merge 12 commits into
masterfrom
add-photography-section

Conversation

@csherland
Copy link
Copy Markdown
Owner

Summary

  • Add site-wide navigation bar with links to Photography and Resume
  • Create photography page at /photography/ with CSS masonry grid (3/2/1 columns responsive) and vanilla JS lightbox
  • Add _data/photography.yml data file for managing photos
  • Exclude docs/ directory from Jekyll build

Details

  • Navigation: Minimal top bar in _layouts/default.html with serif home link + sans-serif page links, styled in _sass/_nav.scss
  • Masonry grid: Pure CSS columns layout — no JS dependencies. Responsive breakpoints at tablet (2 cols) and mobile (1 col)
  • Lightbox: Click-to-open overlay with close via backdrop click, X button, or Escape key. Shows optional title below image
  • Data-driven: Photos managed via _data/photography.yml — currently empty with commented template. Add photos to assets/img/photography/
  • Photography page shows "Photos coming soon." when no photos are configured

Test plan

  • Nav bar appears on all pages with correct links
  • Photography link navigates to /photography/
  • Resume link opens PDF in new tab
  • "Photos coming soon." message displays (no photos configured yet)
  • Dark mode renders correctly for nav and photography page
  • Mobile responsive layout works

🤖 Generated with Claude Code

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Mar 1, 2026

PR Preview Action v1.8.1

QR code for preview link

🚀 View preview at
https://csherland.github.io/pr-preview/pr-8/

Built to branch gh-pages at 2026-03-01 15:00 UTC.
Preview will be ready when the GitHub Pages deployment is complete.

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