Skip to content

Fixed the text on signup page#43

Open
Samiksha-bajoria wants to merge 3 commits intoyashvikram30:mainfrom
Samiksha-bajoria:main
Open

Fixed the text on signup page#43
Samiksha-bajoria wants to merge 3 commits intoyashvikram30:mainfrom
Samiksha-bajoria:main

Conversation

@Samiksha-bajoria
Copy link

@Samiksha-bajoria Samiksha-bajoria commented Jul 25, 2025

I have fixed the text on signup page earlier it was not visually appealing.
Update auth-form.tsx

Fixed #36

followed the code of conduct.

Summary by CodeRabbit

  • Style
    • Refined spacing and line breaks in the authentication form footer for improved readability.
    • Ensures clearer separation between text and the Terms of Service and Privacy Policy links.
    • Reduces awkward wrapping across different screen sizes for a cleaner, more consistent appearance.
    • No changes to functionality, navigation, or content—purely a visual polish.

@vercel
Copy link

vercel bot commented Jul 25, 2025

@Samiksha-bajoria is attempting to deploy a commit to the yash vikram's projects Team on Vercel.

A member of the Team first needs to authorize it.

@yashvikram30
Copy link
Owner

share a screen recording please, in both pc and mobile view, so i can affirm

@yashvikram30
Copy link
Owner

can you submit these changes to "contribution" branch instead of the main branch?

@coderabbitai
Copy link

coderabbitai bot commented Aug 31, 2025

Walkthrough

Adjusted JSX spacing in components/auth/auth-form.tsx CardFooter by inserting explicit {" "} around text near Terms of Service and Privacy Policy links to control line breaks. No logic, state, routing, or export changes.

Changes

Cohort / File(s) Summary
AuthForm spacing tweak
components/auth/auth-form.tsx
Inserted explicit JSX spaces {" "} around text in CardFooter to adjust visual spacing/line breaks; no functional changes.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Assessment against linked issues

Objective Addressed Explanation
Fix sign-up page text so privacy policy line appears on one line [#36]

Poem

I nudged a space with gentle might,
A hop between the words made right.
Now links align, a tidy row—
A bunny’s tweak to make it flow.
With whisker-wisdom, light and spry,
I spaced the clouds across the sky. 🐇✨

Tip

🔌 Remote MCP (Model Context Protocol) integration is now available!

Pro plan users can now connect to remote MCP servers from the Integrations page. Connect with popular remote MCPs such as Notion and Linear to add more context to your reviews and chats.

✨ Finishing Touches
  • 📝 Generate Docstrings
🧪 Generate unit tests
  • Create PR with unit tests
  • Post copyable unit tests in a comment

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
🪧 Tips

Chat

There are 3 ways to chat with CodeRabbit:

  • Review comments: Directly reply to a review comment made by CodeRabbit. Example:
    • I pushed a fix in commit <commit_id>, please review it.
    • Open a follow-up GitHub issue for this discussion.
  • Files and specific lines of code (under the "Files changed" tab): Tag @coderabbitai in a new review comment at the desired location with your query.
  • PR comments: Tag @coderabbitai in a new PR comment to ask questions about the PR branch. For the best results, please provide a very specific query, as very limited context is provided in this mode. Examples:
    • @coderabbitai gather interesting stats about this repository and render them as a table. Additionally, render a pie chart showing the language distribution in the codebase.
    • @coderabbitai read the files in the src/scheduler package and generate a class diagram using mermaid and a README in the markdown format.

Support

Need help? Create a ticket on our support page for assistance with any issues or questions.

CodeRabbit Commands (Invoked using PR/Issue comments)

Type @coderabbitai help to get the list of available commands.

Other keywords and placeholders

  • Add @coderabbitai ignore or @coderabbit ignore anywhere in the PR description to prevent this PR from being reviewed.
  • Add @coderabbitai summary to generate the high-level summary at a specific location in the PR description.
  • Add @coderabbitai anywhere in the PR title to generate the title automatically.

CodeRabbit Configuration File (.coderabbit.yaml)

  • You can programmatically configure CodeRabbit by adding a .coderabbit.yaml file to the root of your repository.
  • Please see the configuration documentation for more information.
  • If your editor has YAML language server enabled, you can add the path at the top of this file to enable auto-completion and validation: # yaml-language-server: $schema=https://coderabbit.ai/integrations/schema.v2.json

Status, Documentation and Community

  • Visit our Status Page to check the current availability of CodeRabbit.
  • Visit our Documentation for detailed information on how to use CodeRabbit.
  • Join our Discord Community to get help, request features, and share feedback.
  • Follow us on X/Twitter for updates and announcements.

Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 0

🧹 Nitpick comments (1)
components/auth/auth-form.tsx (1)

257-259: Spacing fix alone won’t guarantee single line; enforce no-wrap (desktop) and use Next.js Link.

The {" "} additions add spaces but wrapping can still occur. To reliably meet the issue goal, wrap the text and apply responsive no-wrap, and switch to next/link (avoid href="#" jumps).

Apply within these lines:

-        By continuing, you agree to our {" "}
-        <a href="#" className="text-teal-600 hover:underline">Terms of Service</a> and {" "}
-        <a href="#" className="text-teal-600 hover:underline">Privacy Policy</a>.
+        <span className="whitespace-normal md:whitespace-nowrap">
+          By continuing, you agree to our{' '}
+          <Link href="/terms" className="text-teal-600 hover:underline focus-visible:underline">Terms of Service</Link>{' '}and{' '}
+          <Link href="/privacy-policy" className="text-teal-600 hover:underline focus-visible:underline">Privacy Policy</Link>.
+        </span>

Also add the import (outside this hunk):

+import Link from "next/link"

Please confirm with screenshots/recording at mobile and desktop breakpoints, as the maintainer requested, and verify the intended routes for Terms/Privacy. Would you like me to add md:whitespace-nowrap on the CardFooter instead if you prefer no extra span?

📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

💡 Knowledge Base configuration:

  • MCP integration is disabled by default for public repositories
  • Jira integration is disabled by default for public repositories
  • Linear integration is disabled by default for public repositories

You can enable these sources in your CodeRabbit configuration.

📥 Commits

Reviewing files that changed from the base of the PR and between 2066ff2 and 63452cc.

📒 Files selected for processing (1)
  • components/auth/auth-form.tsx (1 hunks)
🔇 Additional comments (2)
components/auth/auth-form.tsx (2)

257-259: LGTM on the explicit JSX spaces.

The inserted {" "} correctly fixes the visual gap between inline elements.


257-259: Process nudge: retarget branch per maintainer comment.

Please change the PR base to the “contribution” branch and attach the requested screen recording for both PC and mobile views.

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.

[Bug] <short description of the issue> fixing the sign up text section

2 participants