Update main.js for each script type to use Plasma 6 API changes #29
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
From my testing, installing this with "Install from File" button in "KWin Scripts" settings doesn't seem to work. Relevant system component versions: |
|
Seems this script might not be necessary for some people anymore. To get my desired behaviour, in "Windows Behaviour", go to "Advanced" tab and set "Virtual Desktop behaviour" dropdown to "Do nothing". The new window of the application I was trying to tame now opens on the currently active desktop.
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Added the fix to support Plasma 6 as described by technewsguy in Issue #27.
Tested the fix on my machine using Fedora KDE Plasma 6 using both KWin Wayland and X11. All three scripts work as expected.
The official Plasma 6 API documentation still hasn't been released yet but luckily other users have already dug into the source code and found the necessary pieces needed to update this script.