Add global animation speed slider to workspace options KCM
ClosedPublic

Authored by davidedmundson on Aug 1 2019, 10:39 PM.

Details

Summary

Introduce a new config entry into kdeglobals AnimationSpeed that will
control /all/ animation speeds in kwin and plasma themes and Qt styles
and kirigami.

The value stored is a multiplier.
i.e 1 == normal speed, 0.5 = twice as fast, 2 = twice as slow.
0 is a special value meaning "disable animations"

The QML is a bit complex to represent a logarithmic slider that goes
4x, 2x, 1x, 0.5x, 0.25x, 0.125x

but still allowing a user to type arbitrary values into their config


Note @kbroulik This is reversed it from what we coded together, because
I decided it was confusing.

Needs relevant patches through the rest of the stack before merging.
Also kwin has some asserts on 0 speed animations that need addressing.

You also need an up-to-date kconfig and qqc2-desktop-style for bugfixes

Test Plan

Looked at config

Diff Detail

Repository
R119 Plasma Desktop
Branch
master
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 14668
Build 14686: arc lint + arc unit
davidedmundson created this revision.Aug 1 2019, 10:39 PM
Restricted Application added a project: Plasma. · View Herald TranscriptAug 1 2019, 10:39 PM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
davidedmundson requested review of this revision.Aug 1 2019, 10:39 PM
davidedmundson updated this revision to Diff 62956.EditedAug 1 2019, 11:42 PM

update

Since we have 4 values on right / left, 0 should be the default one, no?

davidedmundson retitled this revision from Add global animation speed slider to workspace options KCM GNU nano 4.3 /tmp/edit.fd801hu0c34so8k8/new-commit Modified Add global animation speed slider to workspace options KCM to Add global animation speed slider to workspace options KCM.Aug 2 2019, 8:15 AM
broulik added a subscriber: broulik.EditedAug 2 2019, 10:47 AM

Why is it confusing? I find it more confusing if AnimationSpeed=0.5 means *faster* instead of "half the speed" as reading that line would suggest.
However, KWin seems to do it the same way round...

GB_2 added a subscriber: GB_2.Aug 14 2019, 1:15 PM

Why is it confusing? I find it more confusing if AnimationSpeed=0.5 means *faster* instead of "half the speed" as reading that line would suggest.
However, KWin seems to do it the same way round...

I agree.

We agreed that code-wise a multiplier is easier but then the name of the group should be adjusted to reflect that.

rename config option and variable and property names

broulik accepted this revision.Aug 20 2019, 1:50 PM
This revision is now accepted and ready to land.Aug 20 2019, 1:50 PM

Is this going to be able to land before Plasma 5.17 branches in 8 days?

update names as per discussion

This revision was automatically updated to reflect the committed changes.