-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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(icon-button): Fix position when used as TextFieldIcon #4575
base: master
Are you sure you want to change the base?
Conversation
Fixes #4059 This PR fixes the position issue that occurs when an IconButton is used as a TextFieldIcon. It does so by adding negative margin equal to the padding to the IconButton in this particular situation.
I’m closing this pull request for now because of inactivity. @sdomenet we can always reopen it though! :) If you need any help, please feel free to open a GitHub issue. |
fix(text-field-icon): Icon button position
I would like to submit another commit to solve that issue, can I do that here or should I create a new PR? |
Please commit to same PR. Thanks! |
Codecov Report
@@ Coverage Diff @@
## master #4575 +/- ##
=========================================
+ Coverage 98.96% 99.26% +0.3%
=========================================
Files 129 129
Lines 6294 6294
Branches 821 821
=========================================
+ Hits 6229 6248 +19
+ Misses 64 45 -19
Partials 1 1
Continue to review full report at Codecov.
|
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 created some screenshot tests for you. The dense-leading-mdc-icon.html is failing. Please a search for mdc-textfield__icon
under packages/mdc-textfield
. You should also add cases for mdc-icon-button in most/all of those areas.
can we solve PR? |
Fixes #4059
This PR fixes the position issue that occurs when an IconButton is used as a TextFieldIcon.
It does so by adding negative margin equal to the padding to the IconButton in this particular situation.