-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
fixed button size #3194
fixed button size #3194
Conversation
CLA
Hello there and welcome to our project! |
width: 24px; | ||
`; | ||
const StyledBackIconButton = styled(IconButton)` | ||
height: 24px; |
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.
this actually looks unnecessary if you pass the right size to the IconButton (small)
@@ -54,14 +54,23 @@ const StyledTitleContainer = styled.div` | |||
max-width: 50%; | |||
`; | |||
|
|||
const StyledBackIconButton = styled(IconButton)` | |||
const StyledButtonDiv = styled.div` |
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.
I'm a bit confused why we would need a container here.
I'm taking a look!
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.
Thanks for the contribution! While taking a look at your PR, I have actually found ways to simplify a bit more, please take a look at the last commit.
Really appreciate the help, feel free to take any other good first issue!
Fix #2971
The width of the Title was set to 100% which I updated to adjust automatically with enabled and disabled back button. And set the size of buttons to 24px.