GNOME Bugzilla – Bug 696159
Desktop icons move when opening an application
Last modified: 2013-05-22 16:53:58 UTC
After logging into GNOME Shell and opening an application, the icons on the desktop move downwards. I am using git master. OS: Arch Linux
Created attachment 239302 [details] Video
Created attachment 239303 [details] Before
Created attachment 239304 [details] After
-> gnome-shell It looks as if the shell only sets the workarea after the first application launches, for some reason. A quick chat with Jasper on IRC confirmed that, so reassigning to gnome-shell.
if a non-intrusive patch shows up, this would be nice to fix. but not a blocker
is anybody looking into this ?
Created attachment 244838 [details] [review] layout: Fix indentation
Created attachment 244839 [details] [review] layout: Properly order startup initialization In order to make sure that the struts and regions are initialized, we need to make sure that we do with the conditions that nothing is transformed, like the uiGroup, and that everything is visible. These invariants broke during a fix to hide the UI until the system background texture could be loaded. Now, reorder things so that the system background is loaded, and then the UI is properly loaded, and so on.
Review of attachment 244838 [details] [review]: Yep
Review of attachment 244839 [details] [review]: ::: js/ui/layout.js @@ +585,3 @@ + this._startupAnimation(); + return false; + global.stage.show(); Both the emit and the idle_add() should happen inside _prepareStartupAnimation(), I think.
Created attachment 245070 [details] [review] layout: Properly order startup initialization In order to make sure that the struts and regions are initialized, we need to make sure that we do with the conditions that nothing is transformed, like the uiGroup, and that everything is visible. These invariants broke during a fix to hide the UI until the system background texture could be loaded. Now, reorder things so that the system background is loaded, and then the UI is properly loaded, and so on.
Review of attachment 245070 [details] [review]: Looks good.
Attachment 244838 [details] pushed as aefe0d3 - layout: Fix indentation Attachment 245070 [details] pushed as 4b7e230 - layout: Properly order startup initialization