Skip to content

fix: situation selector visibility + case study TOC wrapping#62

Merged
romahawk merged 1 commit intomainfrom
dev
Apr 30, 2026
Merged

fix: situation selector visibility + case study TOC wrapping#62
romahawk merged 1 commit intomainfrom
dev

Conversation

@romahawk
Copy link
Copy Markdown
Owner

Summary

  • Situation selector cards disappearing — removed reveal from the selector section in ServicesPage.jsx; React's className reconciliation was overwriting the reveal--visible class added by IntersectionObserver on every language-switch re-render, collapsing the section to opacity: 0
  • Case study TOC chips cut off — replaced horizontal scroll (overflow-x: auto, hidden scrollbar) with flex-wrap: wrap in modal.css; all "On this page" chips now wrap to additional rows instead of overflowing off-screen
  • Added og-services.png OG image for the /services route

Test plan

  • Open /services, scroll to "Your Situation" — cards visible on first load
  • Switch EN ↔ DE — cards remain visible after language change
  • Open any case study modal on a narrow screen — all "On this page" chips visible (no clipping)
  • Verify modal TOC chips wrap cleanly on desktop and mobile widths

🤖 Generated with Claude Code

- Remove `reveal` from the situation selector section to prevent cards
  disappearing on language-switch re-renders (React className reconciliation
  was stripping the IntersectionObserver-applied `reveal--visible` class)
- Replace horizontal-scroll TOC chip list with flex-wrap so all "On this
  page" chips are visible at any modal width without overflow clipping
- Add og-services.png OG image for the /services route

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@vercel
Copy link
Copy Markdown

vercel Bot commented Apr 30, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
portfolio-fullstack Ready Ready Preview, Comment Apr 30, 2026 7:53am
portfolio-react Ready Ready Preview, Comment Apr 30, 2026 7:53am

@romahawk romahawk merged commit b481c12 into main Apr 30, 2026
4 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