Skip to content

feat: add AudioVerse portfolio page - #75

Merged
narthur merged 2 commits into
masterfrom
feat/audioverse-portfolio-page
Aug 16, 2026
Merged

feat: add AudioVerse portfolio page#75
narthur merged 2 commits into
masterfrom
feat/audioverse-portfolio-page

Conversation

@narthur

@narthur narthur commented Aug 16, 2026

Copy link
Copy Markdown
Owner

What

Adds /audioverse — a portfolio page showing three screenshots of AudioVerse as I built it — and links to it from a new "AudioVerse" entry in the "Things I've Built" list.

Why

The live audioverse.org has diverged from the frontend I built there, so it no longer works as a work sample. This links the project entry to a page I control instead, showing the 2024 captures of the UI I want credit for.

Notes

  • Screenshots downscaled from the originals to 1400px JPEG (12MB → 2MB), with explicit width/height so the browser reserves layout space instead of jumping as they load. First image loads eagerly, the other two lazily.
  • Project links in ProjectList.svelte now only get target="_blank" when the URL is external, so the new internal link stays in-tab.

🤖 Generated with Claude Code

Summary by CodeRabbit

  • New Features
    • Added an AudioVerse project to the portfolio.
    • Added a dedicated AudioVerse project page with project details, accessibility resources, and historical screenshots.
    • Added a responsive, labeled screenshot gallery with improved loading and accessibility information.
    • Added navigation back to the homepage.
    • External project links now open in a new tab, while internal links remain in the current tab.

narthur and others added 2 commits August 16, 2026 08:34
audioverse.org has since diverged from the frontend I built there, so link
the project entry to a page I control showing captures of the work instead.

Screenshots downscaled to 1400px JPEG (12MB -> 2MB). Internal project links
no longer open in a new tab.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Auto-applied low-risk review findings:
- add explicit width/height to screenshots so the browser reserves layout
  space instead of jumping as each loads
- load the first screenshot eagerly; lazy-load the rest
- give both external links on the page target=_blank + rel=noopener
  noreferrer, matching the convention used elsewhere on the site

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@narthur

narthur commented Aug 16, 2026

Copy link
Copy Markdown
Owner Author

Review Loop Summary

Result: clean exit after 2 cycles.

Cycle 1 — static analysis (eslint, gitleaks, semgrep) clean: no secrets, no SAST findings. Three review agents covering CLAUDE.md/knowledge.md compliance, bug scan (commission + omission), comments, git history, security, test coverage, intent reconciliation, and prior PR review feedback.

Findings, all auto-applied as low-risk in 800224e:

Finding Source Fix
Screenshots had no width/height, so the page jumps as each multi-hundred-KB image loads intent recon added intrinsic dimensions to all three
First screenshot was loading="lazy", delaying the one image a visitor actually waits for intent recon first loads eagerly, rest stay lazy
Both external links lacked target="_blank"; one had rel="noopener noreferrer" without a target (inert half) bug scan both now carry target + rel, matching the convention in uses/+page.svelte

No findings from: git history, security, test coverage, prior PR review feedback (no past reviewer comments on these files).

Cycle 2 — reviewed the fix delta only. No findings. Verified the width/height values match the actual files and that intrinsic sizing coexists correctly with the w-full class.

Checks: pnpm lint, pnpm check (320 files, 0 errors), pnpm test, pnpm build all pass locally.

@coderabbitai

coderabbitai Bot commented Aug 16, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository: narthur/coderabbit/.coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: cd6adeed-1400-4d6a-af1c-21bf853d28e3

📥 Commits

Reviewing files that changed from the base of the PR and between 68fd292 and 800224e.

⛔ Files ignored due to path filters (3)
  • static/audioverse/discover.jpg is excluded by !**/*.jpg
  • static/audioverse/home.jpg is excluded by !**/*.jpg
  • static/audioverse/sermon.jpg is excluded by !**/*.jpg
📒 Files selected for processing (2)
  • src/components/ProjectList.svelte
  • src/routes/audioverse/+page.svelte

Included review availability: Your plan includes up to 1 review per rolling hour; 0 remain after this review.


📝 Walkthrough

Walkthrough

Adds an AudioVerse portfolio page with project details, external links, and a responsive screenshot gallery. Adds AudioVerse to the project list and changes link targets so only external HTTP links open in a new tab.

Changes

AudioVerse portfolio

Layer / File(s) Summary
AudioVerse route and gallery
src/routes/audioverse/+page.svelte
Adds page metadata, project content, accessibility-tool and external links, three labeled screenshots, responsive styling, loading behavior, and home navigation.
Project listing integration
src/components/ProjectList.svelte
Adds the AudioVerse project entry and applies target only to links with HTTP URLs. Internal links no longer receive target.

Estimated code review effort: 3 (Moderate) | ~20 minutes

Merge Risk: ⚪ Minimal · up to 80022

This adds a localized AudioVerse portfolio page and keeps its internal project link in the same tab; no actionable merge-blocking risk remains after normal checks and review.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly summarizes the main change: adding the AudioVerse portfolio page.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feat/audioverse-portfolio-page

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@narthur
narthur merged commit 5bccdaa into master Aug 16, 2026
7 checks passed
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