[kstyle] Drop Qt 4 style plugin
ClosedPublic

Authored by zzag on Jan 6 2020, 7:49 PM.

Details

Summary

KWindowSystem cannot be used in the Qt 4 style plugin. On the other hand,
we need to use KWindowSystem to make QWidget-based internal clients in
KWin cast drop-shadows.

Another problem with the Qt 4 style plugin is that some distributions
have already dropped Qt 4, so one has to build it first in order to
verify that his or her change works with Qt 4.

Given maintenance burden and the fact that Qt 6 is around the corner,
this change drops the Qt 4 style plugin.

Diff Detail

Repository
R31 Breeze
Branch
port-to-shadows-api
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 21035
Build 21053: arc lint + arc unit
zzag created this revision.Jan 6 2020, 7:49 PM
Restricted Application added a project: Plasma. · View Herald TranscriptJan 6 2020, 7:49 PM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
zzag requested review of this revision.Jan 6 2020, 7:49 PM
mart added a subscriber: mart.EditedJan 7 2020, 9:31 AM

+1,
not an huge loss for me, qt4 clients are becoming very rare anyways

I don't think this is the right approach as it regresses the Qt4 style.
We can achieve the same thing by having packagers just continue to build an existing old tarball. They keep the shadows, and we can fix Breeze not just for this but other Qt5 porting.

I've emailed plasma and kde-distro-packagers for feedback.

zzag added a comment.EditedJan 7 2020, 10:11 AM

Do you suggest to move qt 4 style plugin to its own repo? I'm not sure that I follow you.

EDIT: never mind, I just saw your email.

zzag planned changes to this revision.Jan 12 2020, 6:59 PM
zzag updated this revision to Diff 73387.Jan 13 2020, 10:37 AM

Drop Qt 4 style plugin

zzag retitled this revision from [kstyle] Remove shadows in Qt 4 style plugin to [kstyle] Drop Qt 4 style plugin.Jan 13 2020, 10:56 AM
zzag edited the summary of this revision. (Show Details)
davidedmundson accepted this revision.Jan 21 2020, 11:37 PM
This revision is now accepted and ready to land.Jan 21 2020, 11:37 PM
This revision was automatically updated to reflect the committed changes.