EliverLara / Sweet-kde

Plasma desktop theme port of my Gtk theme
213 stars 21 forks source link

Plasma 6 migration #48

Open Gigas002 opened 2 months ago

Gigas002 commented 2 months ago

Hello, @EliverLara! Hope you're doing well!

Also calling for @dr460nf1r3 (hope you're not too busy!) as a fellow maintaner for this discussion :)

I've spent some time to help with KDE Plasma 6 theme migration, as this was stale for last several months.

This PR is arguable and contains a lot of breaking changes and propositions and I hope we can discuss it here. I can revert any changes you don't like so don't mind to tell me I'm wrong.

Will close #259, #47 (if I understood it correctly, needs migration to .json format for that), maybe more

So, as I said, I've changed lots of things. Here's a brief review of changes:

Proposition: repo refactoring

As a package maintaner, I find it a bit confusing, that Plasma stuff is incorporated into mostly GNOME/GTK repo. Each time repo update comes, all KDE users recieve it, but 90% of a time it's in fact a gtk theme updates. That's why I propose to rebase all KDE Plasma stuff completely into Sweet-kde.

As it is a breaking change, I think it should be merged into plasma6 branch. nova branch in Sweet repo and master in Sweet-kde will stay unchanged for KDE Plasma 5 users.

Going further into repo refactoring, for the same package maintaining reasons I'd like to propose separating kde/cursors, kde/sddm, kde/Kvantum and kde/konsole each into a separate repo. These projects can be installed and used without dependency on KDE Plasma and pther theme components, so we don't need to force users to keep a track on a whole Sweet-kde theme project. It's a fairly easy change, but will help maintainers and users a lot.

Proposition: theme paths and naming changes

As for minor repo refactoring details, I'd like to address this issue and change paths layout and some naming changes to reflect theme better:

As for Sweet-kde repo:

look-and-feel

cursors

SDDM

Sweet-kde repo

Proposition: change Sweet-kde license to GPLv3

In order to have same license as with GNOME/GTK repo, plus some moved components are licensed under GPLv2/GPLv3 license already

TODO

Useful tips for migration

Gigas002 commented 2 months ago

Ouch, didn't think it'd be THAT big. I can separate this on several PRs in different places, just wanted to have some communication first

dr460nf1r3 commented 2 months ago

Am currently very busy, but will try to review the changes in detail soon. @EliverLara from a superficial look it seems to be a really good enhancement from a maintainers point of view. Would love to have something like this take place. Extracting the KDE theme from the nova branch is of course possible, but it would indeed fit better here.

EliverLara commented 2 months ago

Hi, I really appreciate your contribution and I think it could be a good enhancement for the project to get it separate from gtk-related repo. So trying to give you some feedback:

I find it a bit confusing, that Plasma stuff is incorporated into mostly GNOME/GTK repo

It was done in that way because everything exceptplasma-desktop themeis released under GPL license, the plasma-desktop theme is a fork of a project licenced under CC BY-SA 4.0

I agree with most of the changes you proposed, it is a change that I had been thinking for a long time ago, and as you said it's a movement that could break anything(since kde theming is easily breakable) furthermore there is a lot of files to review so I'd like to do it carefully and if you don't mind I'll try to make the movement this week.

Gigas002 commented 2 months ago

Thank you for the feedback!

the plasma-desktop theme is a fork of a project licenced under CC BY-SA 4.0

Ah, I understand your reasoning now. I think that it's compatible with GPLv3, so I suppose there shouldn't be any breakage in case of re-licensing, if you decide to accept this change ofc

I'd like to do it carefully and if you don't mind I'll try to make the movement this week

Take your time, we're not rushing for these changes and it's perfectly fine to do it step by step. If I can somehow simplify this process for you - please, let me know 🙂

Gigas002 commented 1 month ago

Tried out svg cursors with plasma 6.2.0, seems to work correctly. For the reference: https://blog.vladzahorodnii.com/2024/10/06/svg-cursors-everything-that-you-need-to-know-about-them/