-
Notifications
You must be signed in to change notification settings - Fork 16
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
2580: Center search icon #2599
2580: Center search icon #2599
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.
That solves the issue here very nicely, thank you! But I think we can make the code more future-proof by setting the parent display to flex and aligning its items with the center. That would also keep us safe in case we e.g. change the icon sizing at some point.
sorry i probably should have mentioned it again in the ticket here. This task was about learning sth. about flexbox :) |
I've modified the code and aligned the icon to center from the div parent |
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.
Good Work :) Added a comment about inline-flex
Co-authored-by: Andreas Fischer <37902063+f1sh1918@users.noreply.github.com>
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.
Nicely done!
Short description
The Search Icon is not centered with the input "Suche"
Proposed changes
Center and align the Search Icon with the input, set it a little lower
Side effects
Resolved issues
The Search Icon is aligned with the Input "Suche"
Fixes: #2580