Skip to content

Add Enter/Escape keypress handling to $ionicPopup.prompt() $.alert() and $ionicPopup.confirm() - #4739

Closed
gregallensworth wants to merge 6 commits into
ionic-team:masterfrom
gregallensworth:master
Closed

Add Enter/Escape keypress handling to $ionicPopup.prompt() $.alert() and $ionicPopup.confirm()#4739
gregallensworth wants to merge 6 commits into
ionic-team:masterfrom
gregallensworth:master

Conversation

@gregallensworth

Copy link
Copy Markdown

More detailed description here:
#4737

@jgw96

jgw96 commented Mar 29, 2016

Copy link
Copy Markdown
Contributor

Thanks for the PR! We really appreciate it! I would love to merge this PR but it seems that some of the tests are failing. Would you be interested in fixing those tests and resubmitting the pr?

@jgw96 jgw96 added the needs: reply the issue needs a response from the user label Mar 29, 2016
@jgw96

jgw96 commented Apr 4, 2016

Copy link
Copy Markdown
Contributor

Hello! As it has been a little while since there was any activity on this PR i am going to be closing it, but please feel free to open more PR's with us in the future!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs: reply the issue needs a response from the user

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants