Skip to content

fix(auth): unify OAuth button styles and disable email login#456

Merged
graycyrus merged 1 commit into
tinyhumansai:mainfrom
graycyrus:fix/unified-auth-cta-styles
Apr 9, 2026
Merged

fix(auth): unify OAuth button styles and disable email login#456
graycyrus merged 1 commit into
tinyhumansai:mainfrom
graycyrus:fix/unified-auth-cta-styles

Conversation

@graycyrus
Copy link
Copy Markdown
Contributor

@graycyrus graycyrus commented Apr 9, 2026

Summary

  • Unified all three OAuth CTAs (Google, GitHub, Twitter) to a consistent white pill-button style with light gray border — replacing individual brand-colored backgrounds
  • Updated GitHub and Twitter icon fills to dark colors for visibility on white background
  • Commented out non-functional email login section (input + button) until backend auth flow is implemented
  • Removed unused useState import and email state/handler

Test plan

  • Verify all 3 OAuth buttons render as white pills with consistent styling
  • Verify OAuth login flows still work for Google, GitHub, and Twitter
  • Verify email section is not visible on the Welcome page
  • Run yarn typecheck — passes
  • Run yarn lint — no new warnings

Summary by CodeRabbit

  • Style
    • GitHub and Twitter OAuth sign-in buttons updated to light theme styling
    • Email login option removed; users can now authenticate using OAuth providers only

All three OAuth CTAs (Google, GitHub, Twitter) now share a consistent
white pill-button style with light border instead of individual brand
colors. Email login section commented out as the backend flow is not
yet implemented.
@graycyrus graycyrus merged commit 5dbf9d4 into tinyhumansai:main Apr 9, 2026
6 of 9 checks passed
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Apr 9, 2026

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 4e59b0bb-f877-4f6f-96b6-60c0c7699cec

📥 Commits

Reviewing files that changed from the base of the PR and between 4f0513b and 83f7720.

📒 Files selected for processing (2)
  • app/src/components/oauth/providerConfigs.tsx
  • app/src/pages/Welcome.tsx

📝 Walkthrough

Walkthrough

The PR applies a light theme styling conversion to OAuth provider buttons (GitHub and Twitter) and removes email-based login functionality from the Welcome component, leaving OAuth sign-in as the only authentication method.

Changes

Cohort / File(s) Summary
OAuth Provider Styling
app/src/components/oauth/providerConfigs.tsx
Updated TwitterIcon and GitHubIcon SVG fill colors to darker tones. Converted GitHub and Twitter button styling from dark theme (bg-gray-900, text-white) to light theme (bg-white, text-gray-900) with corresponding hover state adjustments.
Welcome Component Refactoring
app/src/pages/Welcome.tsx
Removed email state management (useState) and form submission logic (handleEmailSubmit). Commented out entire email login UI (divider, form, input fields). Removed !text-xs class from OAuth button. OAuth sign-in is now the only active authentication method.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Possibly related PRs

Poem

🐰 From dark to light, the buttons take flight,
Twitter and GitHub in white now stand bright,
Email login bids its farewell with cheer,
OAuth reigns supreme—simplification is here! ✨

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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

This was referenced Apr 23, 2026
AusAgentSmith pushed a commit to AusAgentSmith/openhuman that referenced this pull request May 23, 2026
…ansai#456)

All three OAuth CTAs (Google, GitHub, Twitter) now share a consistent
white pill-button style with light border instead of individual brand
colors. Email login section commented out as the backend flow is not
yet implemented.
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