kcmlookandfeel - Correctly set the Splash Screen to what the Look & Feel intends for it to be set to
Needs ReviewPublic

Authored by The-Feren-OS-Dev on Mon, Aug 12, 9:45 PM.

Details

Reviewers
davidedmundson
mart
Group Reviewers
Plasma
Summary

BUG: 409015

This is a small change to the Look & Feel KCM Code to fix https://bugs.kde.org/show_bug.cgi?id=409015 by changing the Splash Screen changing code a little bit.
This revision makes Look & Feel check if the package has specified a Splash Screen, that it wants Plasma to change the user's current Splash Screen to, or not via the 'defaults' file and a check for the 'splashrc' 'Theme' value in there. If there isn't a value found it will revert back to the old method of applying Splash Screens - applying the Look & Feel name as the Splash Screen. If a value is found however, ksplashrc's Theme value will be set to that value instead of the Look & Feel's name.

The reason for having the fallback in this code to the old behaviour is that many popular Look & Feels, such as the Adapta Look & Feel, all rely on the fallback method of applying Splash Screens to have their Splash Screens be applied, and if the fallback isn't there nothing will then their currently applied Splash Screen simply won't change at all unless the Look & Feel package specifies a Splash Screen theme.

Diff Detail

Repository
R119 Plasma Desktop
Branch
master
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 15035
Build 15053: arc lint + arc unit
Restricted Application added a project: Plasma. · View Herald TranscriptMon, Aug 12, 9:45 PM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
The-Feren-OS-Dev requested review of this revision.Mon, Aug 12, 9:45 PM
The-Feren-OS-Dev retitled this revision from kcmlookandfeel - Fixed Splash Screen applying to kcmlookandfeel - Correctly set the Splash Screen to what the Look & Feel intends for it to be set to.Mon, Aug 12, 9:48 PM
The-Feren-OS-Dev edited the summary of this revision. (Show Details)Mon, Aug 12, 9:50 PM
The-Feren-OS-Dev edited the summary of this revision. (Show Details)Mon, Aug 12, 9:56 PM
GB_2 added a reviewer: Plasma.Mon, Aug 12, 9:58 PM