Improve portfolio performance and project storytelling - #16
Merged
Conversation
- Media: 5.3MB SENTRY GIF replaced with 244KB WebM (+MP4 fallback, poster, reduced-motion client module, no-CLS dimensions); hero and five other >500KB PNGs converted to WebP (~95% total reduction); originals preserved in source-assets/ (not served) - SENTRY split: V3 page focuses on shipped work with an evidence-based Testing & Verification section and a verified GitHub source link; the V4 roadmap moved to /projects/sentry-v4/ (Concept, disclaimed, cross-linked, listed under a Roadmaps & Concepts index section) - FIRST Robotics: /frc/ rewritten as a ~1,100-word professional summary with an accessible HTML timeline; complete history preserved at /frc/history/; heading hierarchy fixed - Index promise softened to match actual per-page sections; validator updated for grouped navigation markup - BATCH_D_CONTENT_PERFORMANCE_REPORT.md with baseline/after tables and the factual questions still needing Zac Co-Authored-By: Claude Fable 5 <[email protected]>
Contributor
There was a problem hiding this comment.
Pull request overview
This PR updates the Docusaurus-based portfolio site to improve performance and clarity of project storytelling by replacing large media assets with optimized formats, splitting SENTRY V3 (deployed) vs V4 (concept) content into separate pages, and restructuring the FIRST Robotics content into a concise overview plus preserved full history.
Changes:
- Replaces several large PNG/GIF assets with WebP and WebM/MP4 (with poster) and adds reduced-motion handling for autoplay video.
- Adds a new SENTRY V4 roadmap page and updates the projects index to clearly separate “Roadmaps & Concepts” from deployed/prototype projects.
- Refactors FIRST Robotics content into a concise /frc/ page with an HTML/CSS timeline and preserves the full original content in /frc/history/.
Reviewed changes
Copilot reviewed 18 out of 35 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| src/pages/teaching/index.md | Switches ENT260 card image from PNG to WebP. |
| src/pages/teaching/ent260-solidworks.md | Switches ENT260 hero image from PNG to WebP. |
| src/pages/projects/stlink-v3mods.md | Switches SPARK hero image from PNG to WebP. |
| src/pages/projects/sentry-v4.md | Adds new “Concept” roadmap page for SENTRY V4. |
| src/pages/projects/sentry-v3.md | Replaces GIF with video+poster, adds Testing & Verification section, links out to V4 roadmap. |
| src/pages/projects/index.md | Updates project index copy, swaps media, and adds a “Roadmaps & Concepts” section with a V4 card. |
| src/pages/index.md | Switches home hero image to WebP and prioritizes loading. |
| src/pages/frc/history.md | Adds preserved full FIRST Robotics history page. |
| src/pages/frc.md | Rewrites /frc/ into a concise narrative with a semantic HTML timeline and links to full history. |
| src/pages/documentation/selected-essays.md | Switches hero image to WebP and adds intrinsic sizing. |
| src/pages/documentation/scholarship.md | Switches hero image to WebP and adds intrinsic sizing. |
| src/pages/documentation/index.md | Switches documentation landing hero and card images to optimized WebP variants. |
| src/css/custom.css | Adds styling for the FIRST Robotics timeline (and already contains placeholder image styling). |
| src/clientModules/reduced-motion.js | Adds prefers-reduced-motion handling for ambient autoplay videos. |
| source-assets/README.md | Documents preservation of original source media outside the public build. |
| scripts/validate-build.js | Strengthens build validation for server-rendered disclosure links in generated HTML. |
| docusaurus.config.js | Registers the reduced-motion client module. |
| BATCH_D_CONTENT_PERFORMANCE_REPORT.md | Adds a batch report documenting scope, results, and commands used. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Comment on lines
+10
to
+11
| # *FIRST* Robotics Competition | ||
| ## Event Manager & Leadership - *FIRST* Chesapeake |
Comment on lines
+159
to
+161
| background: var(--zc-cyan-500); | ||
| border: 2px solid #fff; | ||
| } |
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
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.
Batch D: ~95% media reduction (SENTRY GIF → WebM/MP4 with poster + reduced-motion handling; six PNGs → WebP; originals preserved in source-assets/), SENTRY V3/V4 split with a Concept-labeled roadmap page, FIRST Robotics condensed to a ~1,100-word summary with an accessible HTML timeline (full history preserved at /frc/history/), evidence-based Testing & Verification on SENTRY, and an accurate project-index promise. Details in BATCH_D_CONTENT_PERFORMANCE_REPORT.md.
Merge after the brand C+D PR (already merged) — CI validates against brand main.
🤖 Generated with Claude Code