-
Notifications
You must be signed in to change notification settings - Fork 2
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Style login page #34
Style login page #34
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Hey! I don't think I have much to add to the review aside from what Chinemerem wrote. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please make the suggested changes prior to merging. Also, after you successfully address a comment you should mark it as resolved.
Co-authored-by: Chinemerem <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry for the late review.
Thank you for the amazing work!!
Notion ticket link
Login Page
Implementation description
SistemaLogoColour
componentindex.tsx
SignInButton
componentSignOutButton
andsession
fromindex.tsx
Steps to test
Desktop Browser UI:
(Dev Tool) Tablet UI:
(Dev Tool) iPhone SE UI:
What should reviewers focus on?
UI alignment with mock
UI transferability (different screen sizes, orientations, etc.)
General code style/best practices
Checklist