Conversation
provsalt
left a comment
provsalt
left a comment
There was a problem hiding this comment.
design doesn't look good to me. the goal was having the fields on the left and having an image on the right. this is completely off from what was discussed.
provsalt
left a comment
provsalt
left a comment
There was a problem hiding this comment.
there seems to be excessive tailwinds classes being used and the icon is using a hardcoded svg value. I'm also not sure what the close banner is there.
There was a problem hiding this comment.
Pull request overview
This PR redesigns the login and signup pages with a modern, responsive UI that replaces the Card component-based layout with a custom split-screen design featuring gradient backgrounds, improved form styling, and mobile-friendly notification banners.
- Replaced Card component layout with a responsive two-column design (form + hero section)
- Added dismissible mobile notification banners for better mobile UX
- Enhanced form inputs with better styling, placeholder text, autocomplete attributes, and improved error messages
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 10 comments.
| File | Description |
|---|---|
| frontend/src/routes/auth/signUp.jsx | Redesigned signup page with split-screen layout, mobile banner, improved form fields with autocomplete, and enhanced error messages |
| frontend/src/routes/auth/login.jsx | Redesigned login page with split-screen layout, mobile banner, improved form fields with autocomplete, and consistent styling with signup page |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
make sure your colours use primary as well. thanks |
No description provided.