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

fix(select): clicking on the input now causes the item list to appear #154

Merged
merged 3 commits into from
Apr 26, 2016

Conversation

jhuntoo
Copy link
Contributor

@jhuntoo jhuntoo commented Apr 20, 2016

This fixed the pre existing issue which I originally noticed, where clicking on the input would not cause the item list to appear

@jhuntoo
Copy link
Contributor Author

jhuntoo commented Apr 20, 2016

The broken build is because there is a dependency on the previous PR

@jhuntoo
Copy link
Contributor Author

jhuntoo commented Apr 25, 2016

@valorkin Rebased on development. CI is now green (admittedly not so comforting yet until tests are added).

@valorkin
Copy link
Member

Looks really good to me!
Next test specs and review https://github.com/valor-software/ng2-select/pull/158/files
if you say it is ok, I will merge

@valorkin valorkin merged commit c46c0ff into development Apr 26, 2016
@valorkin valorkin deleted the fix-click-issue branch April 26, 2016 10:38
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.

2 participants