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

No config options #79

Open
yaroedi opened this Issue Feb 19, 2017 · 36 comments

Comments

Projects
None yet
@yaroedi
Copy link

yaroedi commented Feb 19, 2017

I'm trying to use this script under plasma 5.9.2. I managed to install it and it seems to be working fine, except for the fact that there is no settings option and I can't get into the GUI. Anyone else having this problem?

@drupol

This comment has been minimized.

Copy link

drupol commented Feb 19, 2017

Same here under gentoo.

@faho

This comment has been minimized.

Copy link
Owner

faho commented Feb 19, 2017

I know that there was a similar issue a while back with Fedora that boiled down to wrong packaging.

Do you have any other scripts that do show configuration settings? Do you have /usr/lib/qt/plugins/kwin/effects/configs/kcm_kwin4_genericscripted.so? Or do you have a similarly named file in another directory?

@drupol

This comment has been minimized.

Copy link

drupol commented Feb 19, 2017

I have it in:

 ls -la /usr/lib/qt5/plugins/kwin/effects/configs/kcm_kwin4_genericscripted.so                                                         Mon 20 Feb 2017 12:24:25 AM CET
-rwxr-xr-x 1 root root 563760 Feb 15 10:06 /usr/lib/qt5/plugins/kwin/effects/configs/kcm_kwin4_genericscripted.so*

@faho

This comment has been minimized.

Copy link
Owner

faho commented Feb 19, 2017

That should work - the corresponding line in metadata.desktop is

X-KDE-Library=kwin/effects/configs/kcm_kwin4_genericscripted

So if it is relative to /usr/lib/qt5/plugins in your case, it should be found. I don't know how the default qt plugin path is set, but you could try setting $QT_PLUGIN_PATH to "/usr/lib/qt5/plugins" and then launch kwin_x11 --replace.

@drupol

This comment has been minimized.

Copy link

drupol commented Feb 19, 2017

Nope, it doesn't work...

screenshot_20170220_005622

@yaroedi

This comment has been minimized.

Copy link

yaroedi commented Feb 21, 2017

Same here. Although I kind of worked around it by manually specifying options in ~/.config/kwinrc

@wcerfgba

This comment has been minimized.

Copy link

wcerfgba commented Mar 11, 2017

I too cannot access the config options through 'KWin Scripts'.
I am running openSUSE Tumbleweed, and installed kwin-tiling through the 'KWin Scripts' screen, not from git.
My genericscripted is at /usr/lib64/qt5/plugins/kwin/effects/configs/kcm_kwin4_genericscripted.so.
@yaroedi Could you please share how you configured in kwinrc?

@yaroedi

This comment has been minimized.

Copy link

yaroedi commented Mar 11, 2017

@wcerfgba Sure, here you go:

Under [Plugins], make sure it's enabled with kwin-script-tilingEnabled=true.

And then, for the plugin itself:

[Script-kwin-script-tiling]
fullscreenGaps=true
gapSize=4
screenGapSizeBottom=8
screenGapSizeLeft=8
screenGapSizeRight=8
screenGapSizeTop=8
windowsGapSizeHeight=4
windowsGapSizeWidth=4

But I haven't quite worked out the layouting stuff yet.... ;)

@tillschaefer

This comment has been minimized.

Copy link

tillschaefer commented May 9, 2017

I can also confirm the problem. Running kwin_x11 --replace with the correct plugin path also did not solve the problem. Furthermore, there exist a videowall plugin on my computer for witch the config options are present. It also uses X-KDE-Library=kwin/effects/configs/kcm_kwin4_genericscripted. Thus, I do not think it is a problem with the plugin path.
(btw: also on Gentoo)

KWin spits out these errors here (I do not know if it is related):
file:///home/till/.local/share/kwin/scripts/kwin-script-tiling/contents/code/tilelist.js:353: TypeError: Cannot call method 'readConfig' of undefined

@xenithorb

This comment has been minimized.

Copy link

xenithorb commented May 10, 2017

I too have no config options in Fedora 25 - Plasma 5.9.5

@drupol

This comment has been minimized.

Copy link

drupol commented May 10, 2017

I ended up by disabling it as I couldn't configure it properly 👎

@faho

This comment has been minimized.

Copy link
Owner

faho commented May 10, 2017

So, I have just downloaded the KDE spin of Fedora 25 and started it live in virtualbox. I then installed the script via GHNS, activated it and clicked the config button. The menu appeared.

Is there anything I'm missing? @xenithorb: Did you do anything differently, e.g. install via a different spin? Netinstall?

@tillschaefer:

KWin spits out these errors here (I do not know if it is related):
file:///home/till/.local/share/kwin/scripts/kwin-script-tiling/contents/code/tilelist.js:353: TypeError: Cannot call method 'readConfig' of undefined

Are you using Plasma 5 or 4? That looks like it's 4, which is no longer supported.

@tillschaefer

This comment has been minimized.

Copy link

tillschaefer commented May 10, 2017

I am using the latest stable release of KDE + Qt:

  • Plasma 5.9.5
  • Frameworks 5.33
  • KDE Applications 17.04.0
  • Qt 5.7.1 + 4.8.7 (slotted install)

However, this is an upgrade installation (i.e. Plasma 4 was installed some time ago). Maybe some Plasma 4 configuration persisted after the upgrade?

@xenithorb

This comment has been minimized.

Copy link

xenithorb commented May 10, 2017

Did you upgrade your virtual machine? The live media isn't updated over the course of the release, and by now it has become quite out date. @faho

I have the same Plasma and Kf5 and Qt versions as @tillschaefer, though my Kapps is more out-of-date. This was originally a KDE spin install but it's been updated through 3 versions of Fedora so who knows.

@faho

This comment has been minimized.

Copy link
Owner

faho commented May 10, 2017

Okay, I upgraded my VM and it is indeed broken. The button doesn't even show up. Unfortunately, systemsettings5 in a konsole doesn't print any useful errors, so I have no idea what to do. (At first it prints stuff about not finding the correct .desktop file for the service type, but even after renaming the file it still won't show the menu)

It's possible it's related to Qt5 being 5.7.1 (mine is 5.8.0). Since I don't think anyone here wants to compile their entire desktop anew, we're gonna have to wait on that.

@bahhumbug

This comment has been minimized.

Copy link

bahhumbug commented May 20, 2017

I don't think QT 5.8.0 will solve this as I'm unable to access the configuration using that version (archlinux).

  • plasma 5.9.5
  • plasma framework 5.34
  • qt 5.8.0
@VinegarLove

This comment has been minimized.

Copy link

VinegarLove commented May 24, 2017

Configuration not present for me too.
Same system as bahhumbug

@TomBZombie

This comment has been minimized.

Copy link

TomBZombie commented May 26, 2017

FYI it works on Arch Linux installed via AUR package https://aur.archlinux.org/cgit/aur.git/tree/PKGBUILD?h=kwin-scripts-tiling

@bugworm

This comment has been minimized.

Copy link

bugworm commented May 27, 2017

@TomBZombie this is what I have on arch with package from AUR
image

@TomBZombie

This comment has been minimized.

Copy link

TomBZombie commented May 27, 2017

Interesting, here's what I get:

image

I'm on plasma 5.9.5.

@bugworm

This comment has been minimized.

Copy link

bugworm commented May 27, 2017

@TomBZombie works for me on other user as well, but if you delete it and install again(or create a new user), it won't work anymore

@timlepes

This comment has been minimized.

Copy link

timlepes commented Jun 17, 2017

I am having the same problem described by AnotherBugworm... I get the error "Plugin does not provide configuration file in expected location" when trying to configure it in system settings. I see that there is a file located at /usr/share/kwin/scripts/kwin-scripts-tilling/content/config/main.xml. The video-wall plug-in seems to have a configuration given with the same relative path (...scripts/videowall/content/config/main.xml). The video wall configuration from system settings works (does not give error), but the tiling configuration does not work (gives the error). I am using Arch on x86_64 with kernel 4.11.5-1-ARCH, running KDE 5.35.0 / Plasma 5.10.2. Uninstalling then re-installing the script does not fix the problem. Ideas?

@jonnyrobbie

This comment has been minimized.

Copy link

jonnyrobbie commented Jun 25, 2017

Yeah, chiming in with the same problem. Latest Arch, installed through KDE store the plugin doesn't offer the setting button. Installing the script from AUR, there is a button, but it errors out with that unprovided location message.

@TomBZombie

This comment has been minimized.

Copy link

TomBZombie commented Jun 25, 2017

Try following the instructions in the README.md:

git clone https://github.com/faho/kwin-tiling.git
cd kwin-tiling/
plasmapkg2 --type kwinscript -i .

That worked for me on my laptop. AUR: No config options, installed as above; worked as expected.

@tpnx

This comment has been minimized.

Copy link

tpnx commented Jun 27, 2017

Hi there,

I have the same problems in KDE neon 5.10.2. I tried all the tipps in this Issue, but I am not able to get the config menu running.

ATM I configured the gapps directly in kwinrc. Is there a documentation of all settings?

Regards
tpnx

@dsmorozov

This comment has been minimized.

Copy link

dsmorozov commented Jun 27, 2017

I've faced the issue on latest Arch as well. This fixes the issue in a moment:
cd /usr/share/kservicetypes5 && sudo ln -s kwinscript.desktop kwin-script.desktop

@tpnx

This comment has been minimized.

Copy link

tpnx commented Jun 27, 2017

Thanks,

unfortunately it doesn't work for me.

@BenoitZugmeyer

This comment has been minimized.

Copy link

BenoitZugmeyer commented Jun 27, 2017

I managed to get the configuration button on Arch Linux!

It seems that the kwin script needs to be a known KDE service to get a "Configuration" button (see the condition to show the button here and here). But installing the script via the systemsettings interface or by using the plasmapkg2 utility does not register the script as a KDE service. To do this manually, you have to put the file metadata.desktop in either ~/.local/share/kservices5 or /usr/share/kservices5, example:

mkdir -p ~/.local/share/kservices5
cp ~/.local/share/kwin/scripts/kwin-script-tiling/metadata.desktop ~/.local/share/kservices5/kwin-script-tiling.desktop

I have no idea if there is a way to do this via a command. Also I guess we should open an issue somewhere in the KDE bug tracker but I feel so ignorant how all this should work.

I hope it works for you.

@tpnx

This comment has been minimized.

Copy link

tpnx commented Jun 28, 2017

Works!

Thank you so much!

@drupol

This comment has been minimized.

Copy link

drupol commented Jun 28, 2017

Works too, thanks!

@BoltsJ

This comment has been minimized.

Copy link

BoltsJ commented Jul 9, 2017

Following @BenoitZugmeyer's instructions gets the configuration window to appear, but it is missing the layout and excluded window class sections. Running KDE Neon User Edition.
screenshot_20170709_182026

@faho

This comment has been minimized.

Copy link
Owner

faho commented Jul 12, 2017

Urgh... yeah.

The issue there is that the widget we used (KEditListWidget) has been removed, and AFAIK there's no great replacement. The best we can do is probably QLineEdit, which requires users to enter it as a comma-separated string.

I've now made that change in 3658cc1. This might require that you open up the config UI because (for some reason) this is now stored as class1\\,class2, ie. the "," is backslash-escaped.

@JackKiefer

This comment has been minimized.

Copy link

JackKiefer commented Aug 21, 2017

Running KDE Neon with Plasma version 5.10.4. With the most recent version of the repo I ran into this same issue but was able to fix it per @BenoitZugmeyer 's instructions! :)

@copyme

This comment has been minimized.

Copy link

copyme commented Sep 6, 2017

@faho I think that it would be nice to add the solution of @BenoitZugmeyer into the documentation.

@arusahni

This comment has been minimized.

Copy link
Contributor

arusahni commented Aug 22, 2018

@copyme I filed a PR with @BenoitZugmeyer's fix: #107

@OvermindDL1

This comment has been minimized.

Copy link

OvermindDL1 commented Nov 9, 2018

@BenoitZugmeyer method fixed it here in the latest KDE Neon as well, any chance on having the Makefile copy that file there itself when installing?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment