Differential D25201

Don't set X-Plasma-MainScript for wallpaper plugins
ClosedPublic
Actions

Authored by zzag on Nov 7 2019, 10:00 PM.

Details

Reviewers
davidedmundson
Group Reviewers
Plasma
Commits
R120:cf54a3e77395: Don't set X-Plasma-MainScript for wallpaper plugins
Summary

The file path of the main script is hard-coded to ui/main.qml, so setting
X-Plasma-MainScript is pointless.

Diff Detail

Repository
R120 Plasma Workspace
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
zzag created this revision.Nov 7 2019, 10:00 PM
Restricted Application added a project: Plasma. · View Herald TranscriptNov 7 2019, 10:00 PM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
zzag requested review of this revision.Nov 7 2019, 10:00 PM
zzag edited reviewers, added: Plasma; removed: KWin.Nov 7 2019, 10:03 PM
zzag edited the summary of this revision. (Show Details)Nov 7 2019, 10:05 PM
davidedmundson accepted this revision.Nov 7 2019, 10:37 PM
This revision is now accepted and ready to land.Nov 7 2019, 10:37 PM
Closed by commit R120:cf54a3e77395: Don't set X-Plasma-MainScript for wallpaper plugins (authored by zzag). · Explain WhyNov 18 2019, 8:41 AM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Diff 69906

View Options

wallpapers/color/metadata.desktop

Loading...
View Options

wallpapers/image/imagepackage/metadata.desktop

Loading...
View Options

wallpapers/image/slideshowpackage/metadata.desktop

Loading...