After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 728820 - Auto move windows broken with 3.12
Auto move windows broken with 3.12
Status: RESOLVED FIXED
Product: gnome-shell
Classification: Core
Component: extensions
3.12.x
Other Linux
: Normal major
: ---
Assigned To: gnome-shell-maint
gnome-shell-maint
Depends on:
Blocks:
 
 
Reported: 2014-04-23 21:27 UTC by Anthony Ruhier
Modified: 2014-05-01 11:53 UTC
See Also:
GNOME target: ---
GNOME version: 3.11/3.12


Attachments
auto-move-windows: Fix fallout from API change (1.02 KB, patch)
2014-04-24 10:46 UTC, Florian Müllner
committed Details | Review
workspaceThumbnails: Fix removal of multiple workspaces (1.69 KB, patch)
2014-04-25 00:47 UTC, Florian Müllner
committed Details | Review

Description Anthony Ruhier 2014-04-23 21:27:06 UTC
Hello,
Since Gnome-Shell 3.12, the extension Auto Move Windows doesn't work anymore : it removes all the workspaces and doesn't allow me to create a new one (I am stuck with one workspace, even if I have some opened windows).

It doesn't print any error in the looking-glass. I have tested on 2 computers on ArchLinux, one of these is running on a fresh install (installed directly with gnome 3.12, no upgrade from gnome 3.10).

Thank you !
Comment 1 drago01 2014-04-24 08:43:19 UTC
Please contact the extension auto directly. This is not the appropriate place to file bugs against such extensions.
Comment 2 Florian Müllner 2014-04-24 09:32:48 UTC
(In reply to comment #1)
> Please contact the extension auto directly. This is not the appropriate place
> to file bugs against such extensions.

No. The extension is part of the gnome-shell-extensions module, so this is indeed the appropriate place.
Comment 3 drago01 2014-04-24 10:46:03 UTC
(In reply to comment #2)
> (In reply to comment #1)
> > Please contact the extension auto directly. This is not the appropriate place
> > to file bugs against such extensions.
> 
> No. The extension is part of the gnome-shell-extensions module, so this is
> indeed the appropriate place.

Oh indeed I though this was from e.g.o ... ignore my comment then.
Comment 4 Florian Müllner 2014-04-24 10:46:04 UTC
Created attachment 275036 [details] [review]
auto-move-windows: Fix fallout from API change

MetaWindowActor.get_workspace() was removed, however the now used
MetaWindow.get_workspace() returns a MetaWorkspace rather than an
index, so can't be used directly.

That should fix it.
Comment 5 Florian Müllner 2014-04-24 11:10:17 UTC
Attachment 275036 [details] pushed as fee5495 - auto-move-windows: Fix fallout from API change

Pushing, as the change is obviously correct (and was already reviewed when we fixed the core version of checkWorkspaces())
Comment 6 Anthony Ruhier 2014-04-24 13:53:52 UTC
Many thanks, it works !
Comment 7 Anthony Ruhier 2014-04-24 23:16:11 UTC
I found an other bug in the extension : for example, I have 4 workspaces with one window on each + 1 empty workspace added automatically.
If I close the windows on the 3thrd workspace and then, from the overview, I close the window on the 4th workspace, it will close the workspace 3, 4 and 5 (so it will not keep the usual last empty workspace created dynamically).

I have tried to fix it but I can't find a solution :(
Comment 8 Florian Müllner 2014-04-25 00:46:56 UTC
(In reply to comment #7)
> I found an other bug in the extension

The extension is actually fine now, you found a bug in gnome-shell itself - patch coming :-)
Comment 9 Florian Müllner 2014-04-25 00:47:35 UTC
Created attachment 275091 [details] [review]
workspaceThumbnails: Fix removal of multiple workspaces

When workspaces have been removed, we need to remove the corresponding
thumbnails as well; the number of thumbnails that need removing is
the difference between the old number of workspaces and the new one.
Currently we assume that the old number of workspaces corresponds to
the number of existing thumbnails, but that may actually be wrong:
A thumbnail will still be animated out after its workspace has been
removed. As a result, we end up removing too many thumbnails when a
workspace is removed while a thumbnail of a previously removed workspace
is still animating out. Fix this by basing the old number of workspaces
only on thumbnails that have not been removed previously.
Comment 10 drago01 2014-04-26 15:40:22 UTC
Review of attachment 275091 [details] [review]:

OK.
Comment 11 Florian Müllner 2014-04-26 15:51:10 UTC
Attachment 275091 [details] pushed as 05ddece - workspaceThumbnails: Fix removal of multiple workspaces
Comment 12 Anthony Ruhier 2014-05-01 11:53:16 UTC
Sorry for the late answer, but it works well, thank you a lot for the quick answers and patches !