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

Removed hard-coded pac-input id from input in Autocomplete component. #36

Merged
merged 1 commit into from
May 20, 2021
Merged

Removed hard-coded pac-input id from input in Autocomplete component. #36

merged 1 commit into from
May 20, 2021

Conversation

born2discover
Copy link

@born2discover born2discover commented Apr 12, 2021

Removed a hard-coded #pac-input id from the input in Autocomplete component and replaced it by the ref mechanism as it allows for instantiation of multiple component instances on the same page.

This resolves #35.

@TouchSek
Copy link
Collaborator

@born2discover thank for the PR, I will take a look on this.

Copy link
Collaborator

@TouchSek TouchSek left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@TouchSek TouchSek merged commit 5720671 into chantouchsek:master May 20, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Impossible to instantiate more than one autocomplete component on the same page due to hard set #pac-input
2 participants