Skip to content

Fix : auth UI – add missing CSS, motion, icons, and password strength indicator (#1362).#1474

Open
amankv1234 wants to merge 3 commits into
aryandas2911:mainfrom
amankv1234:fix-auth-ui-issue-1362
Open

Fix : auth UI – add missing CSS, motion, icons, and password strength indicator (#1362).#1474
amankv1234 wants to merge 3 commits into
aryandas2911:mainfrom
amankv1234:fix-auth-ui-issue-1362

Conversation

@amankv1234

Copy link
Copy Markdown

Description

This PR resolves issue #1362.

Problem

  1. Login & Signup inputs lacked visible borders because the Tailwind class input-modern was missing.
  2. UI looked outdated and static.

Solution

  • Added .input-modern and .auth-page-bg utilities with a glass‑morphic look.
  • Implemented floating orb background animations.
  • Integrated Framer Motion staggered animations for form fields.
  • Added Lucide icons to each input.
  • Implemented a password‑strength indicator on the Signup page.
  • Migrated Navbar styles from index.html to CSS.
  • Fixed missing jspdf dependency.

Verification

  • npm run dev runs without console errors.
  • npm run build succeeds (see build logs).
  • Manual UI test: inputs now have visible borders, focus rings, icons, and animations; password strength bar updates as typing.

@amankv1234

Copy link
Copy Markdown
Author

Hi @aryandas2911 ,

The requested changes have been completed. Kindly review and merge the PR if everything is fine.

Thank you!

@aryandas2911

Copy link
Copy Markdown
Owner

lint pls

@amankv1234 amankv1234 force-pushed the fix-auth-ui-issue-1362 branch from 3c8d4db to 8e93854 Compare June 10, 2026 04:22
@amankv1234

Copy link
Copy Markdown
Author

Hi @aryandas2911 ! I've fetched from the main branch, resolved all the merge conflicts, and fixed the linting issues as requested. The build is passing locally as well.

Please review, and let me know if it's good to be merged now! Thanks!

@aryandas2911

Copy link
Copy Markdown
Owner

@amankv1234 resolve conflicts

@aryandas2911

Copy link
Copy Markdown
Owner

resolve conflicts

@amankv1234

Copy link
Copy Markdown
Author

@aryandas2911 , I have resolve all merge conflict take a look and merge it and let me know further any issue

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