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

Comma (,) at the end of Ignore rules breaks/prevents krohnkite from working #223

Open
Likivik opened this issue Jul 30, 2024 · 0 comments
Open
Labels
bug Something isn't working

Comments

@Likivik
Copy link

Likivik commented Jul 30, 2024

Symptom
Comma , at the end of Ignore rules breaks/prevents krohnkite from working - no tiling, no response to shortcuts.
(another symptom is windows' z-index seems to be messed up, meaning some newly opened windows show up below previously active window, which I guess indicates some window management going on 🤔)

How to Reproduce

  1. Open KDE Plasma 6 Settings -> KWin Scripts -> Krohnkite settings -> Rules
  2. Add a comma , to the end of any ignore rule
  3. Apply settings, observe results. (Rebooting, reactivating the script, logging out doesn't fix it)

Expected behavior
Comma is either removed automatically on 'Apply'/'OK' or a warning preventing from Applying with an explanation.

Environment

  • Distro: NixOS 24.11
  • KWin version: 6.1.3
  • Krohnkite version: 0.9.7
  • List of KWin scripts in use: Krohnkite

Notes
Took me a while to diagnose the issue. Not sure how Krohnkite is structured, seems like an issue at the 'reading configuration' stage, if there is such a thing. Perhaps it's a KWin scripting issue, in which case close it as upstream.

P.S.: Huge thanks for the existence of this project!

@Likivik Likivik added the bug Something isn't working label Jul 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant