Skip to content

fix:removing 't' after 14 from login page#339

Open
adityavikram-01 wants to merge 1 commit into
Sachinchaurasiya360:mainfrom
adityavikram-01:fix/typo-t-after-14
Open

fix:removing 't' after 14 from login page#339
adityavikram-01 wants to merge 1 commit into
Sachinchaurasiya360:mainfrom
adityavikram-01:fix/typo-t-after-14

Conversation

@adityavikram-01
Copy link
Copy Markdown

@adityavikram-01 adityavikram-01 commented May 19, 2026

Closes #313

🚀 Description

Fixed a layout typo on the split-screen authentication views on Login Page.

🛠️ Changes Made

  • client/src/module/auth/LoginPage.tsx: Updated object by changing suffix: "t" to an empty string suffix: "".

##Verification (Before vs. After)
before
after

Checklist

  • I have performed a self-review of my own code changes.
  • Only necessary files have been staged and modified.
  • Tested the UI changes locally on a development server.

"Don't worry about the 'login through Gmail' I commented out few code to run it locally and then fixed it back.

Summary by CodeRabbit

  • Bug Fixes
    • Fixed a display issue on the login page where the templates metric was showing an extra character at the end of the count.

Review Change Stack

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented May 19, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 42246c34-356d-433e-a787-2942f522a8c2

📥 Commits

Reviewing files that changed from the base of the PR and between 3552efa and f09fdb8.

📒 Files selected for processing (1)
  • client/src/module/auth/LoginPage.tsx

📝 Walkthrough

Walkthrough

Remove the trailing "t" suffix from the templates count display on the login page. This one-line change updates the promo stat configuration to display only the numeric count without an extra character, matching the styling on the signup page.

Changes

Login Page UI Consistency

Layer / File(s) Summary
Remove trailing 't' suffix from templates metric
client/src/module/auth/LoginPage.tsx
The "templates" promo stat suffix is changed from "t" to an empty string, eliminating the extraneous character appended to the templates count.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

Suggested labels

good first issue, level:beginner, gssoc:approved, type:design

Poem

🐰 A tiny "t" caused quite a fright,
One page had it, one got it right,
Now consistency spreads across the land,
The UI looks perfect—just as planned! ✨

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (4 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly describes the main change: removing the extraneous 't' character after '14' on the login page, which directly addresses the typo fix in the changeset.
Linked Issues check ✅ Passed The pull request successfully addresses issue #313 by removing the stray 't' character from the templates metric suffix, fixing the UI inconsistency on the login page.
Out of Scope Changes check ✅ Passed The change is tightly scoped: only one property in one file was modified to fix the reported typo, with no extraneous changes present.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

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

Warning

Review ran into problems

🔥 Problems

Git: Failed to clone repository. Please run the @coderabbitai full review command to re-trigger a full review. If the issue persists, set path_filters to include or exclude specific files.


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

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

@adityavikram-01
Copy link
Copy Markdown
Author

I have made this contribution under. Please mark it the same.
Thank You

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.

UI inconsistency across sign up and login page

1 participant