Skip to content

Safe cleanup: mobile menu, no-JS fallback, and config cleanup - #123

Merged
dacrab merged 3 commits into
mainfrom
cleanup/safe-pass
Oct 21, 2026
Merged

dacrab merged 3 commits into
mainfrom
cleanup/safe-pass

Conversation

@dacrab

@dacrab dacrab commented Oct 20, 2026 •

Copy link
Copy Markdown
Owner

Summary

  • Remove dead magnetic classes from Header, Footer, and About (no CSS or JS defined them).
  • Make the ci script run only tsc --noEmit; the workflow already runs astro build as a separate step, so the build was running twice.
  • Fix the Vercel ignore rule so it skips renovate/* branches instead of dependabot/*.

Verification

  • bun install --frozen-lockfile
  • bun run ci
  • bun run build

No visual or functional changes.


Additional safe-pass changes

  • Fix the mobile menu offset and Lenis/a11y behavior, add a no-JS fallback and skip link.
  • Add robots/404 noindex handling, contact scroll margin, and clearer footer/link labels.
  • Remove unused dependencies/config/dead CSS and refresh docs.

@coderabbitai

coderabbitai Bot commented Oct 20, 2026 •

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration
  • Configuration used: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: c9c0324e-2a53-4100-8520-d9f00c4a8a67
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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.

@dacrab dacrab changed the title Cleanup: dead classes + duplicate CI build Safe cleanup: mobile menu, no-JS fallback, and config cleanup Oct 21, 2026
@dacrab
dacrab merged commit 4fc4aa7 into main Oct 21, 2026
4 checks passed
@dacrab
dacrab deleted the cleanup/safe-pass branch October 21, 2026 19:06
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