Skip to content
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

Typeahead fields remove leading and trailing spaces #7176

Closed
1 task done
droyad opened this issue Nov 18, 2021 · 3 comments
Closed
1 task done

Typeahead fields remove leading and trailing spaces #7176

droyad opened this issue Nov 18, 2021 · 3 comments
Assignees
Labels
kind/bug This issue represents a verified problem we are committed to solving state/happening Currently in progress (4/4)

Comments

@droyad
Copy link
Contributor

droyad commented Nov 18, 2021

Team

  • I've assigned a team label to this issue

Severity

Annoyance

Version

2021.2.7808

Latest Version

Not applicable

What happened?

When selecting an environment via a variable set, if I type ‘AWS – ‘ it auto removes the last space when searching for the environment.

Reproduction

2021-11-17_15-50-16 (1)

Error and Stacktrace

No response

More Information

No response

Workaround

No response

@droyad droyad added kind/bug This issue represents a verified problem we are committed to solving team/fire-and-motion size/tiny ~1 hour labels Nov 18, 2021
@droyad droyad self-assigned this Nov 21, 2021
@droyad droyad changed the title Variable typeahead field removed trailing space Typeahead fields remove leading and trailing spaces Nov 30, 2021
@droyad
Copy link
Contributor Author

droyad commented Nov 30, 2021

This affects all type ahead combos.

Steps to reproduce:

  1. Create an environment with a space in it
  2. Find a page that has an environment filter
  3. Type the name quickly upto and including the space
  4. The space is removed

This is the line that removes the spaces I think. I'm not 100% sure why it doesn't do it if you type slowly, likely because it doesn't consider whitespace to be a change in of itself.

The typeahead also does not take leading/trailing spaces into account when filtering:
image

@droyad droyad added state/triage and removed kind/bug This issue represents a verified problem we are committed to solving team/fire-and-motion size/tiny ~1 hour labels Nov 30, 2021
@TomPeters TomPeters added state/backlog On the backlog (2/4) and removed state/triage labels Dec 1, 2021
@TomPeters TomPeters added kind/bug This issue represents a verified problem we are committed to solving state/happening Currently in progress (4/4) and removed state/backlog On the backlog (2/4) labels Jan 12, 2022
@TomPeters
Copy link

Release Note: Fixed a bug where whitespace input characters could be trimmed when typing into a MultiSelect

@TomPeters TomPeters assigned TomPeters and unassigned droyad Jan 12, 2022
@Octobob
Copy link
Member

Octobob commented Apr 11, 2022

🎉 The fix for this issue has been released in:

Release stream Release
2021.3 2021.3.11006
2022.1+ all releases

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug This issue represents a verified problem we are committed to solving state/happening Currently in progress (4/4)
Projects
None yet
Development

No branches or pull requests

3 participants