Skip to content
This repository has been archived by the owner on Sep 5, 2024. It is now read-only.

Selection of an item with keyboard recently broken in md-select #4377

Closed
ivan7237d opened this issue Aug 30, 2015 · 1 comment
Closed

Selection of an item with keyboard recently broken in md-select #4377

ivan7237d opened this issue Aug 30, 2015 · 1 comment

Comments

@ivan7237d
Copy link

In current HEAD, if you try to select an item by hitting Enter, the menu won't close and you will get "Uncaught TypeError: Cannot read property 'currentTarget' of undefined":

https://material.angularjs.org/HEAD/#/demo/material.components.select
@shuding shuding mentioned this issue Aug 31, 2015
Closed
@mbarakaja
Copy link

+1
I am having the same issue.

ThomasBurleson pushed a commit that referenced this issue Aug 31, 2015
@shuding @ThomasBurleson
fix(mdSelect): Close menu on hitting Enter key
303ab0d 
call `checkCloseMenu(ev)` with `ev` argument.
ensure references to `ev` are non-null

Fixes #4377. Closes #4384. Closes #4372.
kennethcachia pushed a commit to kennethcachia/material that referenced this issue Sep 23, 2015
@shuding @kennethcachia
fix(mdSelect): Close menu on hitting Enter key
16afa16 
call `checkCloseMenu(ev)` with `ev` argument.
ensure references to `ev` are non-null

Fixes angular#4377. Closes angular#4384.
kennethcachia pushed a commit to kennethcachia/material that referenced this issue Sep 23, 2015
@shuding @kennethcachia
fix(mdSelect): Close menu on hitting Enter key
192ba0c 
call `checkCloseMenu(ev)` with `ev` argument.
ensure references to `ev` are non-null

Fixes angular#4377. Closes angular#4384. Closes angular#4372.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@mbarakaja @ivan7237d