Skip to content

Rebuild homepage with a light editorial theme in place of dark glass UI - #132

Open
TrystonPerry wants to merge 2 commits into
mainfrom
orchestrator/ws-raoi0xx
Open

Rebuild homepage with a light editorial theme in place of dark glass UI#132
TrystonPerry wants to merge 2 commits into
mainfrom
orchestrator/ws-raoi0xx

Conversation

@TrystonPerry

Copy link
Copy Markdown
Contributor

Replaces the dark, glassmorphism visual language across the site with a warm, light editorial theme (inspired by sddesignweek.org), and simplifies several components in the process.

  • app/globals.css: new light color system (--color-bg, --color-ink, --color-accent, etc.), drops the dark prefers-color-scheme override in favor of a fixed light color-scheme, sets body font/colors and link color.
  • app/components/Button.tsx: the three button variants (button/Link/anchor) now share one sharedButtonCss/sharedButtonHoverCss template instead of three duplicated style blocks; pill-shaped buttons, drops the tertiary shimmer/gradient hover effect.
  • app/components/GiveATalkCTA.tsx: replaces a large bespoke glowing/shimmering styled-link (with keyframe animations) with the shared Button component (variant="secondary"); handleClick now tolerates being called without an event.
  • app/components/Card.tsx, app/components/Footer.tsx, app/components/PageContainer.tsx: swap hardcoded dark rgba colors and backdrop-blur for the new CSS variables and bordered surfaces.
  • app/components/Header.tsx: adds a wordmark Logo link (DEVx) next to the menu button, adds aria-label="Open menu" to the menu toggle, and re-themes the sidebar/menu colors to the light palette.
  • app/page.tsx: largest change — removes PotionBackground/ErrorBoundary and the fixed dark BackgroundContainer; replaces the three separate "CONNECT/INSPIRE/BUILD" full-viewport scroll-triggered sections with a single pillars array rendered as a PillarsGrid of cards; the about section drops the full-bleed slider-with-overlay layout for a two-column text/photo grid; organizer cards and the "Join us" section are restyled with the new light theme; useInView calls now pass once: true so animations don't replay on scroll.
  • app/layout.tsx: widens the Chivo Google Font weight range (adds 500/600) to support the new UI's font-weight usage.

Flagged for review: this is a broad, mostly non-functional visual rewrite (dark theme -> light theme, glass/blur -> flat/bordered), and the homepage's scroll-triggered feature sections were replaced with a static grid, changing that part of the page's interaction/animation behavior.

No tests were added or run as part of this diff.


Shipped from an orchestrator chat workspace (orchestrator/ws-raoi0xx).

@TrystonPerry

TrystonPerry commented Aug 17, 2026

Copy link
Copy Markdown
Contributor Author

🤖 Automated review — PR #132

Critical / high-priority findings:

No changes to header nav links' target/rel — those are pre-existing. Let me verify the Header's mobile menu external links to be thorough about pre-existing rel handling, but since the diff doesn't touch them, not introduced by this PR.

I've completed a thorough review. Let me also confirm the final page.tsx compiles into valid JSX by verifying the build isn't broken via the type-check (already passed).

Let me also double-check the page.tsx organizers section — OrganizerCard uses rel="noopener noreferrer" which is good. The OrganizerCard is styled.a with as={motion.a}. Fine.

Let me make sure I haven't missed anything in the truncated page.tsx diff — particularly checking that no data-fetching/error-handling logic was altered in a way that breaks the next-event fetch. I saw the loadNextEvent logic is identical. Good.

Conclusion: This PR is a comprehensive visual/theme redesign. Type-check passes. No critical correctness bugs or security holes introduced. The pre-existing HeroSocialIcon/header external links missing rel="noopener" are not introduced by this PR.

I'll report NO FINDINGS.

Actually, wait. Let me reconsider the sharedButtonCss / sharedButtonHoverCss interpolation once more, since this is genuinely the trickiest part and if wrong it would break all buttons visually (but not a security/correctness issue, and type-check + it's just CSS). Even if it didn't render correctly, that's a visual bug, not critical/high correctness. And it does render correctly.

Let me also verify there's no issue with the Button commonProps spreading $variant/$size onto the DOM element. Since the styled components declare these as transient props (starting with $), styled-components strips them from the DOM. Good — no React DOM warnings/errors.

Final: NO FINDINGS.


orchestrator session ddi8ale · openrouter:deepseek/deepseek-v4-flash-0731

— posted by the workspace's reviewer agent

Temporary reference images for PR #132 review — not part of the app.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@TrystonPerry

TrystonPerry commented Aug 17, 2026

Copy link
Copy Markdown
Contributor Author

🤖 Automated review — PR #132

No findings.


orchestrator session qo9mz5h · openrouter:deepseek/deepseek-v4-flash-0731

— posted by the workspace's reviewer agent

@TrystonPerry

Copy link
Copy Markdown
Contributor Author

Screenshots — current state of every page

Captured from a local dev run against this branch (commit 2ada119). Header/Footer are redesigned everywhere (light theme, DEVx wordmark); only the homepage has had its full content redesigned so far. Everything else below still renders with the old dark/shader theme in the page body — that's the planned next pass, not a bug.

✅ Redesigned

Home /
home

🔜 Not yet redesigned (old dark theme, new header/footer)

Events /events
events

Event detail /events/[eventId]
event-detail

Watch /watch
watch

Talk /talk
talk

Submit a talk /submit-talk
submit-talk

Login /login
login

Setup /setup
setup

Reset password /reset-password
reset-password

Whois /whois
whois

Nametags /nametags
nametags

Event terms /event-terms
event-terms

Slides /slides
slides

Slide detail /slides/[slug]
slide-detail

Doorbell kiosk /doorbell
doorbell

Admin: talks /admin/talks (unauthenticated view)
admin-talks

Admin: thumbnails /admin/thumbnails (unauthenticated view)
admin-thumbnails


The .github/pr-previews/redesign/ images in this commit are just for this review — safe to delete before merge.

🤖 Generated with Claude Code

@TrystonPerry

Copy link
Copy Markdown
Contributor Author

This is a test PR experiment. Ignore

@samholmes

Copy link
Copy Markdown
Contributor

Are you experimenting with how far you can one-shot a redesign? 😉

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.

2 participants