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 662457 - Workspaces list in the Activities overlay should always be expanded when there are windows on other workspaces
Workspaces list in the Activities overlay should always be expanded when ther...
Status: RESOLVED FIXED
Product: gnome-shell
Classification: Core
Component: general
unspecified
Other All
: Normal minor
: ---
Assigned To: gnome-shell-maint
gnome-shell-maint
: 665906 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2011-10-22 13:39 UTC by Greg K Nicholson
Modified: 2014-05-14 19:58 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
workspaces-view: Adjust heuristic for hiding workspace switcher (1.90 KB, patch)
2011-10-22 16:27 UTC, Florian Müllner
needs-work Details | Review
overviewControls: Adjust heuristic for hiding workspace switcher (1.86 KB, patch)
2013-09-06 23:40 UTC, Florian Müllner
committed Details | Review

Description Greg K Nicholson 2011-10-22 13:39:12 UTC
Presently, the workspaces list in the Activities overview is shown expanded if there is more than one workspace with a window on it.

The intention is to expose thee workspaces list only if workspaces are being used. The present heuristic is suboptimal when there are exactly two workspaces and the second workspace is selected.

Instead, the workspaces list should be expanded if there is a window on any workspace other than the selected workspace.

This is equivalent to collapsing the list only when there are exactly two workspaces, and workspace 1 is selected (and the mouse isn't hovering the area).

Steps to reproduce:

1. Open three windows each on their own workspace.
2. Go to the third workspace.
3. Close the window on this workspace.

You're taken back to the Activities overview, which shows there are no windows on this workspace. The workspaces list at the right is expanded, so it's easy to see there are windows on other workspaces.

4. Select the second workspace.
5. Select the window on it and pretend to use the window for a few seconds.
6. Close the window.

You're taken back to the Activities overview, which shows there are no windows on this workspace. However, the workspaces list is collapsed, because there are only two workspaces.
Comment 1 Florian Müllner 2011-10-22 16:27:38 UTC
Created attachment 199726 [details] [review]
workspaces-view: Adjust heuristic for hiding workspace switcher

(This will need some comments and a proper commit message if we want to include it)


(In reply to comment #0)
> The intention is to expose thee workspaces list only if workspaces are being
> used. The present heuristic is suboptimal when there are exactly two workspaces
> and the second workspace is selected.

True. I doubt we can find a heuristic which is optimal in every single case, so it's all about weighing different drawbacks and make a pick.


> Instead, the workspaces list should be expanded if there is a window on any
> workspace other than the selected workspace.

I attached a patch which implements the suggested behavior, but I'm not convinced it is actually better. It certainly fixes the case described, but in the case of two workspaces, it also causes the workspace switcher to pop in/out when changing workspaces, which is rather confusing (at least to me).

Let's get some designer input on this ...
Comment 2 Florian Müllner 2011-12-10 11:51:46 UTC
*** Bug 665906 has been marked as a duplicate of this bug. ***
Comment 3 Allan Day 2011-12-12 09:22:14 UTC
(In reply to comment #1)
> Created an attachment (id=199726) [details] [review]
> workspaces-view: Adjust heuristic for hiding workspace switcher
> 
> (This will need some comments and a proper commit message if we want to include
> it)
> 
> 
> (In reply to comment #0)
> > The intention is to expose thee workspaces list only if workspaces are being
> > used. The present heuristic is suboptimal when there are exactly two workspaces
> > and the second workspace is selected.
> 
> True. I doubt we can find a heuristic which is optimal in every single case, so
> it's all about weighing different drawbacks and make a pick.
> 
> 
> > Instead, the workspaces list should be expanded if there is a window on any
> > workspace other than the selected workspace.
> 
> I attached a patch which implements the suggested behavior, but I'm not
> convinced it is actually better. It certainly fixes the case described, but in
> the case of two workspaces, it also causes the workspace switcher to pop in/out
> when changing workspaces, which is rather confusing (at least to me).
> 
> Let's get some designer input on this ...

I quickly tried the patch. It feels like an improvement to me, since it helps you to know where you are and gives you an easy way to go return to the first workspace. Maybe I'm missing something though.
Comment 4 Owen Taylor 2012-02-06 17:24:39 UTC
Review of attachment 199726 [details] [review]:

The behavior seems OK to me - likely an improvement - but it seems to break workspace switch animations, at least going between the first workspace and the second empty workspace, which makes it confusing what is going on.
Comment 5 Jasper St. Pierre (not reading bugmail) 2012-09-28 00:00:18 UTC
This was implemented.
Comment 6 Florian Müllner 2012-09-28 00:20:44 UTC
No, it wasn't.
Comment 7 Florian Müllner 2013-09-06 23:40:04 UTC
Created attachment 254321 [details] [review]
overviewControls: Adjust heuristic for hiding workspace switcher

Rebased to master, in case we want to pick this up.
Comment 8 Florian Müllner 2014-04-30 17:33:35 UTC
Attachment 254321 [details] pushed as 50f0fc4 - overviewControls: Adjust heuristic for hiding workspace switcher
Comment 9 Greg K Nicholson 2014-05-14 19:58:39 UTC
Woo! Thankyou for fixing this!

(I have to use OS X at work and every time I try to switch workspaces I'm always reminded how much easier Gnome is.)