-
Notifications
You must be signed in to change notification settings - Fork 2.8k
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
fix(ui): fix styling on sign up and reset screens #7301
fix(ui): fix styling on sign up and reset screens #7301
Conversation
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.
nice! looking good, just a quick question then let's do this thing
.ant-input-affix-wrapper-status-error:not(.ant-input-affix-wrapper-disabled):not(.ant-input-affix-wrapper-borderless).ant-input-affix-wrapper, | ||
.ant-input-affix-wrapper-status-error:not(.ant-input-affix-wrapper-disabled):not(.ant-input-affix-wrapper-borderless).ant-input-affix-wrapper:hover { | ||
background-color: transparent; | ||
} |
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.
woah haha is there a simpler way to set this background color?
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.
It has been simplified!
fullName: string; | ||
email: string; | ||
password: string; | ||
confirmPassword: string; | ||
title: string; |
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.
just curious why the type casting here and above is necessary?
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.
TS was throwing an error for me, I think because it can't detect that the styled Form.Item is no longer a Form.Item
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.
ah cool beans
6811448
to
cec4716
Compare
cec4716
to
53f0071
Compare
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.
woo!
Screenshots
Before
![image](https://user-images.githubusercontent.com/19418814/217896669-3ae3d78d-5a32-44d9-8d76-b4640275e878.png)
![image](https://user-images.githubusercontent.com/19418814/217897560-c9ae9c06-20f7-497f-aac4-b671f8b9dbe9.png)
After
![image](https://user-images.githubusercontent.com/19418814/217897328-9d12844d-0d3e-4da7-a483-b15f4b9416c3.png)
![image](https://user-images.githubusercontent.com/19418814/217897624-5632b6c0-9fa5-4545-8f79-24455a545d0b.png)
Checklist