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

[FEATURE REQUEST] Possibility to change ng-trim attribute of the inputs #2064

Open
VitaliyKim opened this issue Oct 14, 2017 · 3 comments · May be fixed by #2157
Open

[FEATURE REQUEST] Possibility to change ng-trim attribute of the inputs #2064

VitaliyKim opened this issue Oct 14, 2017 · 3 comments · May be fixed by #2157

Comments

@VitaliyKim
Copy link

VitaliyKim commented Oct 14, 2017

Hi all!

Need to handle SPACE characters.
Angular trims it and does not call refresh in case SPACE entered, because ng-trim is true by default for inputs.
Is it possible to give control over this attribute somehow?

Thank you for the great tool!

@VitaliyKim VitaliyKim changed the title Possibility to change ng-trim attribute of the inputs [FEATURE REQUEST] Possibility to change ng-trim attribute of the inputs Oct 14, 2017
@Jefiozie
Copy link
Contributor

Hi @VitaliyKim,

As described in the issue template we would like to have a full description with plunker and version.
In the case we can have a look at the issue and reproduce the issue much quicker.
At the moment I don't understand what the request is so please clarify?

@glutentag
Copy link

glutentag commented Aug 6, 2018

Hello,

I believe I have the same issue. I guess what @VitaliyKim said is it would be nice to have the option to not trim the input.

For instance, with the candidates:

"foo"
"  bar"
"    baz"

I would like to search and display the white spaces. For the candidate list and for the selected candidate, it can be done with CSS. But for the input, we currently have to customize the template, adding an option to specify the ng-trim value would be better.

Here is a plunkr showing that, with version 0.19.8.

glutentag pushed a commit to glutentag/ui-select that referenced this issue Aug 29, 2018
@glutentag glutentag linked a pull request Aug 29, 2018 that will close this issue
glutentag pushed a commit to glutentag/ui-select that referenced this issue Aug 29, 2018
@kamwoz
Copy link

kamwoz commented Nov 14, 2018

needed feature, thumb up

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

Successfully merging a pull request may close this issue.

4 participants