Skip to content
This repository was archived by the owner on Sep 5, 2024. It is now read-only.

A vertical scrollbar appeared on inputs with icon with no padding-bottom #2609

Closed
jgoux opened this issue Apr 29, 2015 · 7 comments
Closed
Assignees
Milestone

Comments

@jgoux
Copy link

jgoux commented Apr 29, 2015

Hi,
since the last release, a vertical scrollbar appeared on inputs with icon when the input-container has no padding-bottom.
Example : http://codepen.io/jgx/pen/mJyZoZ
Usually, this is useful when the input is alone in its container or is the last one.

@jgoux jgoux changed the title vertical scrollbar appeared on inputs with icon with no padding-bottom A vertical scrollbar appeared on inputs with icon with no padding-bottom Apr 29, 2015
@ghost
Copy link

ghost commented Apr 29, 2015

Your pen doesn't display the scrollbar. However I have the same issue.

I fixed it by doing padding-bottom:4px on the input-container.

@ThomasBurleson
Copy link
Contributor

Does this Commit SHA f1df6dc not fix this now?

@ThomasBurleson ThomasBurleson added this to the 0.9.0 milestone Apr 30, 2015
@ThomasBurleson ThomasBurleson self-assigned this Apr 30, 2015
@jgoux
Copy link
Author

jgoux commented Apr 30, 2015

@ThomasBurleson According to the codepen, it seems good now 👍

@jgoux jgoux closed this as completed Apr 30, 2015
@jgoux
Copy link
Author

jgoux commented Apr 30, 2015

Sorry I didn't see @robertbaker 's answer, I reopen the issue waiting for his feedback.

@jgoux jgoux reopened this Apr 30, 2015
@ThomasBurleson
Copy link
Contributor

@jgoux - I will close for now. Can reopen if problem persists.

@ghost
Copy link

ghost commented Apr 30, 2015

It is fixed for me. However the icon isn't vertically centered in an md-toolbar
capture

<md-input-container md-no-float class="navbar-search">
    <md-icon md-svg-icon="ic_search_24px"></md-icon>
    <input class="search-input" type="text" 
           placeholder="Search"
           ng-model="search.text" 
           ng-model-options="{ debounce: 700 }"
           ng-blur="inputBlur()"
           ng-focus="inputFocus()">
</md-input-container>

@ThomasBurleson
Copy link
Contributor

@robertbaker - do you have quick demo that we can use to easily test and iterate a good fix?
Post on #2597 if/when you do. Thank you.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants