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

Add windowquit option and get tests working #306

Merged
merged 2 commits into from
Feb 13, 2021

Conversation

aerusso
Copy link
Contributor

@aerusso aerusso commented Oct 21, 2020

Send a _NET_WINDOW_CLOSE event to a given window. This allows for a graceful window shutdown, as opposed to the existing windowclose, which is too rude, and windowkill, which is too lethal.

Send a _NET_WINDOW_CLOSE event to a given window.  This allows for a
graceful window shutdown, as opposed to the existing windowclose, which
is too rude, and windowkill, which is too lethal.

Signed-off-by: Antonio Russo <aerusso@aerusso.net>
assert_not_equal is not defined in minitest. Provide it in
xdo_test_helper.rb.

Signed-off-by: Antonio Russo <aerusso@aerusso.net>
@aerusso aerusso changed the title Add windowquit option Add windowquit option and get tests working Oct 22, 2020
@jordansissel
Copy link
Owner

Code looks good, thanks!

I tested with xdotool selectwindow windowquit and it worked as expected. 👍

@jordansissel jordansissel merged commit ffe0d73 into jordansissel:master Feb 13, 2021
buldi pushed a commit to buldi/xdotool that referenced this pull request Apr 23, 2022
* Add windowquit option

Send a _NET_WINDOW_CLOSE event to a given window.  This allows for a
graceful window shutdown, as opposed to the existing windowclose, which
is too rude, and windowkill, which is too lethal.

Signed-off-by: Antonio Russo <aerusso@aerusso.net>

* Add assert_not_equal ruby test function

assert_not_equal is not defined in minitest. Provide it in
xdo_test_helper.rb.

Signed-off-by: Antonio Russo <aerusso@aerusso.net>
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