Skip to content
This repository was archived by the owner on Oct 2, 2019. It is now read-only.

Wrong inputWidth, input too long #2010

Open
NaturalL opened this issue Jun 9, 2017 · 3 comments
Open

Wrong inputWidth, input too long #2010

NaturalL opened this issue Jun 9, 2017 · 3 comments

Comments

@NaturalL
Copy link

NaturalL commented Jun 9, 2017

Bug description: Wrong inputWidth, the inputWidth shoud minus the container border-width or padding with different theme.

-          var inputWidth = containerWidth - input.offsetLeft - 10;
+          var inputWidth = containerWidth - input.offsetLeft;

Link to minimally-working plunker that reproduces the issue:

http://plnkr.co/edit/OAemHKmhjJmB0iF2lMX0

UI-Select: Version: 0.19.8

@Jefiozie
Copy link
Contributor

Jefiozie commented Jun 9, 2017

I think you are refering to this line 839 but i don't see the - 10.

Please share some more info. Thanks.

@NaturalL
Copy link
Author

@Jefiozie This commit 7ad4ef1 remove "- 10", but I think "- 10" is necessary.

@aurelienlt
Copy link

Already known issue: #1980
But strangely it has never been fixed.

DmitryGonchar added a commit to DmitryGonchar/ui-select that referenced this issue Jun 17, 2017
input is not too big or too small - it takes the whole remaining part of the current row.
If smaller than the min input width - goes to the next row

closes angular-ui#1980, possibly angular-ui#2010
DmitryGonchar added a commit to DmitryGonchar/ui-select that referenced this issue Jun 17, 2017
input is not too big or too small - it takes the whole remaining part of the current row.
If smaller than the min input width - goes to the next row

closes angular-ui#1980, possibly angular-ui#2010
aurelienlt pushed a commit to aurelienlt/ui-select that referenced this issue Jun 30, 2017
simon-n pushed a commit to simon-n/ui-select that referenced this issue Aug 22, 2017
…tiple" control (missing a valid sourcemap because of wrecked build environment)
simon-n pushed a commit to simon-n/ui-select that referenced this issue Aug 22, 2017
…dth for "multiple" control (missing a valid sourcemap because of wrecked build environment)
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

3 participants