add "all" to "set_desktop_for_window"#40
Conversation
|
I think instead of another command, we can just change "set_desktop_for_window" to accept "all" as the argument. (Since it already accepts "current_desktop", it's trivial to modify the parser for "all".) In the window menu, "All desktop" is a radio button along with other desktop radio buttons, thus it should be a special desktop number argument to align with the UI. I'm considering dropping KDE5 support, so we can focus on KDE6 in this PR. |
fae1620 to
fae1690
Compare
Makes sense - I've done that now 👍
Well that makes it easier for me! I just made it output an error if it's used in KDE5. |
Fixes #39
I haven't tested KDE5 yet, unsure if the
window.onAllDesktopsproperty exists there. I'll check and report back.