Bug 477092

Summary:	ExpandableListItem open/close animations do not follow systemwide animation speed
Product:	[Frameworks and Libraries] libplasma	Reporter:	Catty Steve <cattysteve89265>
Component:	components	Assignee:	Plasma Bugs List <plasma-bugs>
Status:	RESOLVED FIXED
Severity:	minor	CC:	me, nate, notmart, uwuwotisdis
Priority:	NOR	Keywords:	qt6
Version:	5.245.0
Target Milestone:	---
Platform:	Arch Linux
OS:	Linux
Latest Commit:		Version Fixed In:	6.1
Sentry Crash Report:

Description Catty Steve 2023-11-16 12:33:41 UTC

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
***


STEPS TO REPRODUCE
1. Open Network/Removable Devices in System Tray
2. click on "expand" button
3. change animation speed in System Settings and do 1~2 again

OBSERVED RESULT
The speed of "Expand" animation doesn't change.

EXPECTED RESULT
The speed of "Expand" animation should respect animation speed setting.

SOFTWARE/OS VERSIONS
Operating System: Arch Linux 
KDE Plasma Version: 5.27.80
KDE Frameworks Version: 5.245.0
Qt Version: 6.6.0
Kernel Version: 6.6.1-zen1-1-zen (64-bit)
Graphics Platform: Wayland
Processors: 16 × AMD Ryzen 7 6800U with Radeon Graphics
Memory: 13.3 GiB of RAM
Graphics Processor: AMD Radeon Graphics
Manufacturer: LENOVO
Product Name: 82QF
System Version: Yoga 14c ARB7

ADDITIONAL INFORMATION

Comment 1 Nate Graham 2023-11-16 21:53:07 UTC

Found the issue, patch incoming soon.

Comment 2 Bug Janitor Service 2023-11-16 21:57:07 UTC

A possibly relevant merge request was started @ https://invent.kde.org/plasma/plasma-framework/-/merge_requests/975

Comment 3 Nate Graham 2024-03-05 22:03:33 UTC

*** Bug 482493 has been marked as a duplicate of this bug. ***

Comment 4 Nate Graham 2024-03-24 22:50:05 UTC

Git commit 796177d05f3339b5f6847665c99be218c2566fe3 by Nate Graham.
Committed on 24/03/2024 at 22:47.
Pushed by ngraham into branch 'master'.

applet: Set a duration for the ExpandableListItem's transitions

This way it's under our control and also follows the systemwide
animation speed setting.

M  +2    -2    applet/contents/ui/ConnectionListPage.qml

https://invent.kde.org/plasma/plasma-nm/-/commit/796177d05f3339b5f6847665c99be218c2566fe3

Comment 5 Nate Graham 2024-03-24 22:50:08 UTC

Git commit 0d2ba75f20455b1ec0519eb9c8ac3ce9e623f20e by Nate Graham.
Committed on 24/03/2024 at 22:48.
Pushed by ngraham into branch 'master'.

Set a duration for the ExpandableListItem's transitions

This way it's under our control and also follows the systemwide
animation speed setting.

M  +2    -2    plasma/package/contents/ui/main.qml

https://invent.kde.org/plasma/plasma-vault/-/commit/0d2ba75f20455b1ec0519eb9c8ac3ce9e623f20e

Comment 6 Nate Graham 2024-03-24 22:50:10 UTC

Git commit a3f126bc07ac33668c5212437519098fc614ecf4 by Nate Graham.
Committed on 24/03/2024 at 22:48.
Pushed by ngraham into branch 'master'.

applet: Set a duration for the ExpandableListItem's transitions

This way it's under our control and also follows the systemwide
animation speed setting.

M  +2    -2    src/applet/package/contents/ui/FullRepresentation.qml

https://invent.kde.org/plasma/bluedevil/-/commit/a3f126bc07ac33668c5212437519098fc614ecf4

Comment 7 Nate Graham 2024-03-24 22:50:13 UTC

Git commit d638345144fb9697fb1a3845a79142f3ec41f871 by Nate Graham.
Committed on 24/03/2024 at 22:48.
Pushed by ngraham into branch 'master'.

applet: Set a duration for the ExpandableListItem's transitions

This way it's under our control and also follows the systemwide
animation speed setting.

M  +2    -2    src/plasmoid/package/contents/ui/FullRepresentation.qml

https://invent.kde.org/plasma/print-manager/-/commit/d638345144fb9697fb1a3845a79142f3ec41f871

Comment 8 Nate Graham 2024-03-24 22:50:16 UTC

Git commit 10578ba5caad3b747c927eef60e002fefba4eb9f by Nate Graham.
Committed on 24/03/2024 at 22:47.
Pushed by ngraham into branch 'master'.

applets/devicenotifier: Set duration for the ExpandableListItem transitions

This way it's under our control and also follows the systemwide
animation speed setting.

M  +3    -3    applets/devicenotifier/package/contents/ui/FullRepresentation.qml

https://invent.kde.org/plasma/plasma-workspace/-/commit/10578ba5caad3b747c927eef60e002fefba4eb9f

Comment 9 Nate Graham 2024-03-24 22:50:42 UTC

That should do the trick for all of them. This is now fixed in Plasma 6.1.