Skip to content

Fix/gallery yaml front matter#2

Merged
chengjiahao1234 merged 5 commits intomainfrom
fix/gallery-yaml-front-matter
Dec 3, 2025
Merged

Fix/gallery yaml front matter#2
chengjiahao1234 merged 5 commits intomainfrom
fix/gallery-yaml-front-matter

Conversation

@chengjiahao1234
Copy link
Collaborator

Type of Change

Please select the type of change (uncomment one):

  • ✅ Bug Fix
  • ✅ Enhancement / New Feature
  • ✅ Styling / CSS

Summary

Briefly describe what this PR changes:

This PR fixes the gallery page 404 error, resolves image overlap issues, and implements homepage design improvements including institutional logos and a more compact header navigation.

Changes Made

List specific files or sections changed:

  • _pages/gallery.md - Fixed YAML front matter delimiter (---- to ---)
  • _layouts/gallerylay.html - Added bottom spacing to prevent footer overlap
  • _sass/_gallery.scss - Fixed grid-row-end initialization to prevent image overlap
  • js/gallery-masonry.js - Removed duplicate code causing initialization issues
  • _pages/home.md - Replaced carousel with institutional affiliation logos (Concordia, McGill, Mila)
  • _layouts/homelay.html - Removed lab logo and "Welcome our website!" text
  • css/main.scss - Increased header logo size (40px→60px) and reduced navbar height (5.5rem→2.75rem)

Testing Checklist

Please confirm you've tested the following:

  • [] Changes display correctly on desktop (1920x1080 or similar)
  • [] Changes display correctly on tablet (768x1024 or similar)
  • [] Changes display correctly on mobile (375x667 or similar)
  • [] Tested in light theme
  • [] Tested in dark theme
  • [] No broken links or images
  • [] No console errors
  • [] Ran bundle exec jekyll serve locally without errors

Screenshots (if applicable)

Gallery Page: Fixed 404 error - page now loads correctly with proper image spacing and no footer overlap

Homepage: Replaced carousel with clean institutional logos (Concordia, McGill, Mila) centered below vision statement

Header: Larger logo (60px) with reduced navbar height for more compact, professional appearance

Related Issues

Fixes gallery page 404 error
Fixes image overlap in gallery masonry layout
Improves homepage branding and header usability

Additional Notes

All changes maintain responsive design across screen sizes. The gallery masonry layout now properly calculates row spans without overlapping images. Homepage presents a cleaner, more professional appearance with institutional branding. Header is more compact while maintaining readability.


By submitting this PR, I confirm:

  • [] I have tested these changes locally
  • [] I have followed the code standards in DEVELOPMENT.md
  • [] I have updated documentation if needed
  • [] I am ready for this to be reviewed and merged

@chengjiahao1234 chengjiahao1234 merged commit bcce14e into main Dec 3, 2025
1 check passed
@chengjiahao1234 chengjiahao1234 deleted the fix/gallery-yaml-front-matter branch February 5, 2026 22:05
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