Created attachment 150626 [details] "Use" button after installing color schemes SUMMARY *** NOTE: If you are reporting a crash, please try to attach a backtrace with debug symbols. See https://community.kde.org/Guidelines_and_HOWTOs/Debugging/How_to_create_useful_crash_reports *** New transition effect is not triggered when changing theme through knewstuff STEPS TO REPRODUCE 1. install a color scheme from the "Get New Color Schemes" button in system settings 2. click 'Use' after installing OBSERVED RESULT Color schemes changes without transition EXPECTED RESULT A transition effect like the one you get when changing from the Colors kcm SOFTWARE/OS VERSIONS Linux/KDE Plasma: 5.25.2 (available in About System) KDE Plasma Version: 5.25.2 KDE Frameworks Version: 5.96.0 Qt Version: 5.15.5 ADDITIONAL INFORMATION
Can confirm. We need to manually trigger the Blend effect when this button is clicked.
(In reply to Nate Graham from comment #1) > Can confirm. We need to manually trigger the Blend effect when this button > is clicked. The button internally calls plasma-apply-colorscheme https://invent.kde.org/plasma/plasma-workspace/-/blob/master/kcms/colors/plasma-apply-colorscheme.cpp#L59 Not sure why it doesn't trigger the effect, since the same function is called from the KCM https://invent.kde.org/plasma/plasma-workspace/-/blob/master/kcms/colors/colors.cpp#L71
(In reply to Kareem from comment #2) > (In reply to Nate Graham from comment #1) > > Can confirm. We need to manually trigger the Blend effect when this button > > is clicked. > > The button internally calls plasma-apply-colorscheme > https://invent.kde.org/plasma/plasma-workspace/-/blob/master/kcms/colors/ > plasma-apply-colorscheme.cpp#L59 > > Not sure why it doesn't trigger the effect, since the same function is > called from the KCM > https://invent.kde.org/plasma/plasma-workspace/-/blob/master/kcms/colors/ > colors.cpp#L71 It gets done in void KCMColors::save(), in the KCM code. So we will need to add similar code to do that in either plasma-apply-colorscheme, or in the knewstuff code that calls it. I think it would probably make sense to do it in plasma-apply-colorscheme.
Fixed as of https://invent.kde.org/plasma/plasma-workspace/-/commit/08d37c535ea1438e124347a4988bff0eb98f375d in plasma-workspace.