Dev

[MODSetter]

Description

fix: update GoogleLoginButton and HeroSection components

Motivation and Context

FIX #

Screenshots

API Changes

• This PR includes API changes

Change Type

• Bug fix
• New feature
• Performance improvement
• Refactoring
• Documentation
• Dependency/Build system
• Breaking change
• Other (specify):

Testing Performed

• Tested locally
• Manual/QA verification

Checklist

• Follows project coding standards and conventions
• Documentation updated as needed
• Dependencies updated as needed
• No lint/build errors or new warnings
• All relevant tests are passing

High-level PR Summary

This PR makes minor UI adjustments to the login and homepage components. The hero section's "Get Started" button now redirects to /login instead of /contact, and a large block of descriptive text in the Google login page has been commented out to simplify the interface.

⏱️ Estimated Review Time: 5-15 minutes

💡 Review Order Suggestion

Order	File Path
1	surfsense_web/components/homepage/hero-section.tsx
2	surfsense_web/app/(home)/login/GoogleLoginButton.tsx

Summary by CodeRabbit

• Updates
  • Removed the animated notice banner from the login interface
  • Updated the primary button navigation to redirect to the login page

[vercel]

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

Project	Deployment	Preview	Comments	Updated (UTC)
surf-sense-frontend	Building	Preview	Comment	Oct 31, 2025 9:40am

[coderabbitai]

Caution

Review failed

The pull request is closed.

Walkthrough

Two UI components are modified: the animated notice banner in the Google login button is commented out, and the homepage hero section's primary call-to-action button target is changed from "/contact" to "/login".

Changes

Cohort / File(s)	Summary
Login Component UI surfsense_web/app/(home)/login/GoogleLoginButton.tsx	Animated notice banner (yellow card) is commented out; Google login button and handler logic remain unchanged
Homepage Navigation surfsense_web/components/homepage/hero-section.tsx	Primary CTA button navigation target changed from "/contact" to "/login"

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

• No logic changes; purely UI/navigation adjustments (commenting out and string replacement)
• Changes are isolated and independent of each other

Possibly related PRs

• fix: recommended fixes #354: Modifies the homepage primary CTA button styling and label alongside the navigation target changes in this PR
• chore: suggested changes #441: Also modifies surfsense_web/components/homepage/hero-section.tsx and comments out CTA elements
• chore: qol patches #70: Inverse operation—adds the animated banner to GoogleLoginButton that this PR comments out

Poem

🐰 The banner fades to silence bright,
While homepage paths now steer toward login's light,
A gentle click, a route refined,
Two simple tweaks, two minds aligned!

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch dev

---

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 9ce00ca and e86c818.

📒 Files selected for processing (2)

• surfsense_web/app/(home)/login/GoogleLoginButton.tsx (2 hunks)
• surfsense_web/components/homepage/hero-section.tsx (1 hunks)

---

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

• X
• Mastodon
• Reddit
• LinkedIn

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

[recurseml]

Review by RecurseML

🔍 Review performed on 9ce00ca..e86c818

✨ No bugs found, your code is sparkling clean

✅ Files analyzed, no issues (2)

• surfsense_web/app/(home)/login/GoogleLoginButton.tsx
• surfsense_web/components/homepage/hero-section.tsx