Skip to content
This repository was archived by the owner on Oct 8, 2021. It is now read-only.

Esc (Escape key) should close dialogs #3107

Closed
deanrad opened this issue Nov 18, 2011 · 4 comments
Closed

Esc (Escape key) should close dialogs #3107

deanrad opened this issue Nov 18, 2011 · 4 comments

Comments

@deanrad
Copy link

deanrad commented Nov 18, 2011

To comply with the defacto convention ...

@eddiemonge
Copy link
Contributor

This really only applies to desktops since mobile devices don't normally have the esc key. Because of that, I think its unneeded complexity because then it also asks, "what about click outside the dialog?"

@deanrad
Copy link
Author

deanrad commented Nov 18, 2011

Good point. And what about click outside the dialog? User-configurable
setting on launching the dialog ?

On Fri, Nov 18, 2011 at 3:42 PM, Eddie Monge Jr.
reply@reply.github.com
wrote:

This really only applies to desktops since mobile devices don't normally have the esc key. Because of that, I think its unneeded complexity because then it also asks, "what about click outside the dialog?"


Reply to this email directly or view it on GitHub:
#3107 (comment)

@toddparker
Copy link
Contributor

I think both are good ideas. Could you please link this up to the feature request wiki page and we'll re-open once we decide to add this feature?

@deanrad
Copy link
Author

deanrad commented Nov 21, 2011

Done - thanks for letting me know what to do !

On Sun, Nov 20, 2011 at 10:00 PM, Todd Parker
reply@reply.github.com
wrote:

I think both are good ideas. Could you please link this up to the feature request wiki page and we'll re-open once we decide to add this feature?


Reply to this email directly or view it on GitHub:
#3107 (comment)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants