Disabling the animations for the Breeze Widget style in systemsettings makes moving panels in dolphin, marble impossible. Kdenlive even crashes when moving panels. Reproducible: Always Steps to Reproduce: 1. Disable animation for breeze widget in systemsettings 2. Open Dolphin and unlock sidepanels if locked 3. Try to move the panels Actual Results: Panels snap back to their initial position Expected Results: Panels should snap to new position Tested on Maui and Netrunner Core aswell as KDE Neon 5.8.2
I can also confirm this issue with Oxygen with disabled animations.
Hi, Thanks for reporting. However I cannot reproduce I suspect a bug in Qt, in fact. (which I do not have in my version). The only thing disabling animations does to panels in breeze and oxygen is change the returned value of "SH_Widget_Animate" if you have access to breeze sources, can you apply the following patch and see if code still crashes ? diff --git a/kstyle/breezestyle.cpp b/kstyle/breezestyle.cpp index 8a02d75..fa98a7b 100644 --- a/kstyle/breezestyle.cpp +++ b/kstyle/breezestyle.cpp @@ -646,7 +646,8 @@ namespace Breeze case SH_Menu_SloppySubMenus: return true; #if QT_VERSION >= 0x050000 - case SH_Widget_Animate: return StyleConfigData::animationsEnabled(); + // case SH_Widget_Animate: return StyleConfigData::animationsEnabled(); + case SH_Widget_Animate: return true; case SH_Menu_SupportsSections: return true; #endif
... or rather: not crashes, but: if you can again move panels?
Yes this fixes the issue. If it is a bug in Qt then the version might be important 5.7.0 is what all 3 distros use.
Thanks for testing. So it is an upstream bug indeed (the only thing the patch does is change the value of a flag which is used internally by Qt and by Qt only. The version of Qt I have is 5.7.0 (and I do not have the bug). what is yours ?
It is also 5.7.0 coming from the Neon packages.
ok. Not sure what to do about it then ... at least report the bug upstream (Qt) something like: "setting SH_Widget_Animate to false makes Sidepanel moving impossible". Will you do that ? Here at the breeze/oxygen level, the only thing I can do is apply the patch (and force animation of sidepanels all the time disregarding the actual option), but that would be a regression (triggered by upstream ...) Wdyt ?
Reported to Qt here: https://bugreports.qt.io/browse/QTBUG-56736
Thanks ! I'll leave this bug open in the meanwhile, waiting for further developments.
Cannot reproduce with Qt 5.7.1 from openSUSE's Tumbleweed repositories, so it might already be fixed in Qt. I only tested Dolphin and kdenlive with Breeze animations disabled.
Yes, see QTBUG-54185. I suggest to add this information to the Qt bug you reported.
for me also doesn't happen anymore on dolphin built against qt 5.7 branch, so post 5.7.0