Skip to content
This repository was archived by the owner on Apr 1, 2020. It is now read-only.

The App doesn't quit with [command+Q] on macOS #928

Closed
a-eid opened this issue Nov 13, 2017 · 5 comments
Closed

The App doesn't quit with [command+Q] on macOS #928

a-eid opened this issue Nov 13, 2017 · 5 comments

Comments

@a-eid
Copy link

a-eid commented Nov 13, 2017

Most if not all macOS applications terminate with [Command + Q].

@dasosjt
Copy link

dasosjt commented Nov 13, 2017 via email

@ghost
Copy link

ghost commented Nov 13, 2017

Strange. I'm on macOS with version 0.2.16 and it does quit on cmd+Q when I try it.

@bryphe
Copy link
Member

bryphe commented Nov 13, 2017

Thanks for the report, @a-eid !

Strange, this should have been addressed by PR #787 (there is actually configuration for it here:

input.bind("<m-q>", "oni.quit")
)

I wonder if there is a certain case where the input bindings aren't getting captured? That could potentially be a more general issue impacting other scenarios..

@a-eid @dasosjt - does this reproduce consistently for you?

In addition, did you use the latest release binary (0.2.16), or build from source?

@bryphe
Copy link
Member

bryphe commented Dec 4, 2017

I'll close this out since I'm unable to reproduce. Feel free to re-open or log a new issue if you're still seeing this!

@bryphe bryphe closed this as completed Dec 4, 2017
@benjamin-rood
Copy link

benjamin-rood commented Dec 24, 2017

Hi @bryphe I can reproduce this bug, but only under a distinct condition: If one closes the Oni window, with :q for example, then command+Q will no longer work. (Otherwise +Q is working correctly.)

Logically, if the only Oni Window/tab is closed, then this should result in also quitting the app.

Sidenote: I think q should behave the same as bd, closing the currently open tab, rather than closing all the tabs. i.e. behave the same as closing a split inside NeoVim?

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

4 participants