search

fennerm / flashfocus

Simple focus animations for tiling window managers.
MIT License
751 stars 9 forks source link

cli '-l' short parameter overloaded with log level and flash lone windows #54

Closed marty-oehme closed 4 years ago

marty-oehme commented 4 years ago

On the current master branch, cli.py defines the short option for both flashing lone windows and for setting the log level output to -l (here).

That means currently, loglevel overrides lone windows setting -- when doing e.g. flashfocus -l never it will complain that never is not a valid option for loglevels.

Perhaps reverting the loglevel short parameter to its previous 'verbosity' -v might be an option?

Thank you for the awesome software fennerm!

fennerm commented 4 years ago

Oof not my best moment. That seems like a reasonable solution to me, putting a patch release together.

fennerm commented 4 years ago

Alright should be fixed in v2.2.1 @marty-oehme. Thanks for reporting!