Closing ParaView causes corrupted settings on Linux with Breeze theme
It does not seem to impact everyone on Linux though.
How to reproduce:
- Delete existing settings
- Launch ParaView master (note that it is not reproducible on ParaView 5.6 release or build)
- Close it properly (killing it do not corrupt the settings)
- Launch it again, all docked widgets and toolbars are closed
My specs:
- Arch Linux
- KDE with Breeze theme
- Qt 5.12
- Show closed items
Activity
-
Newest first Oldest first
-
Show all activity Show comments only Show history only
- Author Contributor
@lrineau reproduces this issue on :
- Fedora 28
- KDE
- Qt 5.11
- Michael Migliore changed the description
changed the description
- Author Contributor
@mwestphal does not reproduce on:
- Arch Linux
- Awesome
- Qt 5.12
- Author Contributor
I do not reproduce on:
- Kunbuntu 18.04
- KDE
- Qt 5.9
Edited by Michael Migliore It seems related to KDE Plasma.
If I run paraview with:
XDG_CURRENT_DESKTOP=Gnome ./bin/paraview
even inside my KDE session, then the problems disappear.
I had a look at
/proc/$(pidof paraview)/maps
, withXDG_CURRENT_DESKTOP=Gnome
and without, and the difference seems:@@ -1,3 +1,4 @@ +/usr/lib64/libdbusmenu-qt5.so.2.6.0 /usr/lib64/qt5/plugins/imageformats/kimg_eps.so /usr/lib64/qt5/plugins/imageformats/kimg_exr.so /usr/lib64/qt5/plugins/imageformats/kimg_pcx.so @@ -19,5 +20,6 @@ /usr/lib64/qt5/plugins/imageformats/libqwebp.so /usr/lib64/qt5/plugins/platforminputcontexts/libcomposeplatforminputcontextplugin.so /usr/lib64/qt5/plugins/platforms/libqxcb.so -/usr/lib64/qt5/plugins/platformthemes/libqgtk3.so +/usr/lib64/qt5/plugins/platformthemes/KDEPlasmaPlatformTheme.so +/usr/lib64/qt5/plugins/styles/breeze.so /usr/lib64/qt5/plugins/xcbglintegrations/libqxcb-glx-integration.so
In one of those two extra Qt5 plugins, there must be one that triggers the issue.
- Maintainer
@michael.migliore : can you had a screenshot of the issue ?
- Michael Migliore changed the description
changed the description
- Michael Migliore changed title from Closing ParaView 5.6 causes corrupted settings on Linux to Closing ParaView causes corrupted settings on Linux
changed title from Closing ParaView 5.6 causes corrupted settings on Linux to Closing ParaView causes corrupted settings on Linux
- Michael Migliore changed the description
changed the description
- Mathieu Westphal (Kitware) changed title from Closing ParaView causes corrupted settings on Linux to Closing ParaView causes corrupted settings on KDE
changed title from Closing ParaView causes corrupted settings on Linux to Closing ParaView causes corrupted settings on KDE
- Maintainer
I do reproduce with switching to KDE instead of Awesome.
- Maintainer
it is caused by breeze.so, removing it or removing its permissions resolve the issue.
I've opened an issue there: https://bugs.kde.org/show_bug.cgi?id=402244
Next step would be to test with a minimal Qt application, but I'm going to pass on that.
- Michael Migliore changed title from Closing ParaView causes corrupted settings on KDE to Closing ParaView causes corrupted settings on Linux with Breeze theme
changed title from Closing ParaView causes corrupted settings on KDE to Closing ParaView causes corrupted settings on Linux with Breeze theme
- Michael Migliore changed the description
changed the description
- Author Contributor
Indeed, it's not related with KDE, I do reproduce with i3 and Breeze theme set with qt5ct.
It's definitely related to Breeze. - Michael Migliore changed the description
changed the description
- Author Contributor
Another note: it is related to the
Layout
property undermainWindow
group. Removing this line fixes the issue (but removes user custom layout of course). - Michael Migliore mentioned in merge request !2963 (merged)
mentioned in merge request !2963 (merged)
- Michael Migliore assigned to @michael.migliore
assigned to @michael.migliore
- Kitware Robot closed via merge request !2963 (merged)
closed via merge request !2963 (merged)
- Kitware Robot added workflow:needs-verification label
added workflow:needs-verification label
- Maintainer
@mwestphal Since you could replicate this issue (at least on KDE), could you test a nightly, and if this fix is correct, remove the "needs-verification" label? Thanks! Alan
- Maintainer
This bug actually does not impact the release or nightly as Qt is shipped with it and it does not contain breeze.so.
- Author Contributor
That also means that the release does not use Qt theme of the desktop.
- Maintainer
shipping breeze.so with ParaView would require us to build some part of KDE in the superbuild, I'm not sure we want to do that.
- Mathieu Westphal (Kitware) removed workflow:needs-verification label
removed workflow:needs-verification label