mkropat / jumpapp

A run-or-raise application switcher for any X11 desktop
MIT License
639 stars 46 forks source link

try _NET_CLIENT_LIST if _NET_CLIENT_LIST_STACKING doesn't work #62

Closed MaxGyver83 closed 3 years ago

MaxGyver83 commented 3 years ago

In window managers that don't support _NET_CLIENT_LIST_STACKING (dwm), jumpapp emit this error message:

/usr/bin/jumpapp: line 248: [[: found.: syntax error: invalid arithmetic operator (error token is ".")

With this PR, jumpapp queries _NET_CLIENT_LIST when _NET_CLIENT_LIST_STACKING doesn't contain any values. This fixes #54 in a sense that this error message doesn't occur anymore.

But it still doesn't work as expected. When you run jumpapp firefox:

When firefox is not active, jumpapp brings you always to the firefox window created first. (Without this PR, the firefox window created last.) Maybe dwm doesn't update _NET_CLIENT_LIST. Nevertheless, I think this PR brings an improvement to dwm users.

MaxGyver83 commented 3 years ago

I have tested it on another PC and now I get the error message again. I need to analyze why it happens.

gitlerat commented 3 years ago

Hi do you have any news on this? It would be really cool to use dwm with jumpapp :)

MaxGyver83 commented 3 years ago

For me it works well with the limitation that I can't toggle between windows of the same application. But I use only one Firefox window and one terminal window (with tmux/byobu) anyway. If this is okay for you, you can simply clone/download my bugfix branch: https://github.com/MaxGyver83/jumpapp/tree/feature/better-dwm-support