Feature/login modal#181
Conversation
…avatar integration in main website
|
@Kirti-Pant is attempting to deploy a commit to the Tanmay Kalra's projects Team on Vercel. A member of the Team first needs to authorize it. |
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
|
@TanmayKalra09 is it not merged yet?? |
|
@TanmayKalra09 kindly merge my PR so that I can get credits of my task done. |
|
Hey @Kirti-Pant, thank you for your efforts, we really appreciate the work you’ve done. |
|
@TanmayKalra09 will I get my credits?? |
Yes of course you will get |
|
@TanmayKalra09 let me know if Login button is working or not. |
|
@Tanziha-Sheikh and @Unnati-coder25 kindly merge my PR. |
|
@TanmayKalra09 kindly merge my PR so that I can get my credits. |
|
@TanmayKalra09 kindly reply to my messages |
|
@Kirti-Pant Please resolve the conflicts |
Thank you for your contribution. We can only proceed with merging the PR if it is working perfectly and meets all required criteria. If there are any issues or incomplete features, unfortunately, we won’t be able to merge it until those are fully addressed. Please ensure everything is functioning as expected so we can move forward. |
|
@Kirti-Pant Are you working on resolving the conflicts? |
|
@TanmayKalra09 yes I'm working on it. |
|
@TanmayKalra09 kindly authorize my resolved files. |
|
@TanmayKalra09 I resolved the conflicts but still Vercel wants authorization to deploy kindly help me out with this if something has to be done from your end make sure to do it so that my PR can get merged. |
|
Conflicts are not resolved yet. Please resolve the following files: package-lock.json, package.json, and src/App.jsx. |
|
@TanmayKalra09 I've resolved the files and did all commits I don't why still showing conflicts. |
|
There are conflicts, which is why this is showing. Please resolve the merge conflicts first. |
|
@TanmayKalra09 I'll try it again and let you know as my code is working properly and there are no errors in files opened in VS Code. |
|
@TanmayKalra09 I created a new PR kindly look into it as it is showing no conflicts |
2 participants
🚀 Pull Request
📋 Description
This PR adds a modern, accessible Login modal component to the project and integrates it into the main website.
It includes form validation, a show/hide password toggle, Gravatar avatar integration, post-login user greeting, animated SVG branding, and full accessibility support.
The modal redirects the user to the homepage upon successful login, enhancing the authentication UX across the site.
Key highlights:
Implements Login.tsx as a reusable modal with clean React hooks structure.
Includes Login.css for styling, animations, and responsive design.
Adds Layout.css for site-wide layout consistency (if used).
Integrates Gravatar avatars, toast notifications, and visual feedback.
Ensures accessible form fields, labels, ARIA roles, and screen-reader friendly content.
🔍 Type of Change
Bug fix 🐞
✅ New feature ✨
✅ Documentation update 📝
Refactoring or code improvement ♻️
Other (please describe):
🙋 Your Details
🧪 How Has This Been Tested?
Describe how you tested your changes.
Manually tested end-to-end by opening the Login modal, entering various email/passwords, checking password validation and show/hide toggle, submitting the form, and verifying:
The greeting and Gravatar avatar display on successful login.
Toast notifications correctly appear for success and errors.
The modal automatically closes and redirects to the homepage.
Checked accessibility with keyboard navigation and screen reader (labels, ARIA, focus management).
📸 Screenshots (if applicable)
Before:
No Login Page
After:
Login Component Added:
✅ Checklist