search

peterfajdiga / karousel

Scrollable tiling Kwin script
GNU General Public License v3.0
286 stars 5 forks source link

Incompatibility Issue with KDE Plasma 6 #29

Closed Riebeck-one closed 8 months ago

Riebeck-one commented 10 months ago

Hi,

Karousel appears to be incompatible with KDE Plasma 6 RC1. Are there plans for an update to ensure compatibility with the final release of Plasma 6?

peterfajdiga commented 10 months ago

Hi, thanks very much for the heads-up! It will be updated, but I don't know how soon. At the latest a few weeks after the stable release of KDE Plasma 6.

peterfajdiga commented 8 months ago

Porting is in progress, but there's a lot of fixing that needs to be done. If anyone knows of a list of breaking changes in the Kwin scripting API, please let me know.

Riebeck-one commented 8 months ago

I don't know much about it, but I could find those two links that could help : https://develop.kde.org/docs/plasma/kwin/ https://invent.kde.org/plasma/kwin/-/wikis/Porting-your-scripts-to-KWin-6

peterfajdiga commented 8 months ago

Thanks. I've already read these, but they're unfortunately both quite outdated/incomplete.

But regardless, I've had some time this weekend and I think I've got it working. I'll test it for about a week to see if there's still any issues before creating a new release. For the impatient, you can try the kwin6 branch.

Riebeck-one commented 8 months ago

Oh nice ! I think I'll try it tomorrow. Would you like to get some feedback about it if I find bugs ?

peterfajdiga commented 8 months ago

Yes, please :)

Riebeck-one commented 8 months ago

I think I installed it correctly. I can turn it on in the Kwin script menu and adjust the settings. However, it doesn't seem to change anything. Windows aren't tiled, either by default or when I use the keyboard shortcut (meta + space).

It might be because of an installation issue. To install it, I just copied everything from the package folder into ~/.local/share/kwin/scripts/karousel.

peterfajdiga commented 8 months ago

It needs to be built first, running make install will take care of it. For your convenience, here's a built package: karousel_kwin6_wip.tar.gz

Riebeck-one commented 8 months ago

Thanks ! Got it working. I'll keep you updated if I find any issue.

baduhai commented 8 months ago

Looking good on Plasma 6, haven't run into any issues so far.

baduhai commented 8 months ago

So I've run into a few issues, which I think are ultimately the same. The workspace switching popup gets tiled along with other windows. Same issue with the panel editing tools and alt-tab popup, they also get tiled when they shouldn't.

peterfajdiga commented 8 months ago

@baduhai I've fixed the issue with the popups getting tiled. If you notice any more windows that are tiled, but shouldn't be, please open an issue.

peterfajdiga commented 8 months ago

Version 0.8 supports Plasma 6 (and only Plasma 6).