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 694092 - Clipping is not reset correctly when workspace switcher is hidden
Clipping is not reset correctly when workspace switcher is hidden
Status: RESOLVED FIXED
Product: gnome-shell
Classification: Core
Component: overview
unspecified
Other Linux
: Normal normal
: ---
Assigned To: gnome-shell-maint
gnome-shell-maint
Depends on:
Blocks:
 
 
Reported: 2013-02-18 14:52 UTC by Giovanni Campagna
Modified: 2013-09-14 16:32 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
WorkspacesView: set the clip when setting the geometry (1.02 KB, patch)
2013-03-10 18:50 UTC, Giovanni Campagna
none Details | Review

Description Giovanni Campagna 2013-02-18 14:52:02 UTC
See attached screencast.

This happens only if you have two workspaces when you enter the overview, and you move your windows to have one, which hides the chooser.
Comment 1 Giovanni Campagna 2013-02-18 14:54:21 UTC
The attachment was too big for bugzilla, so the screencast is temporarily hosted at http://people.gnome.org/~gcampagna/Screencast%20da%2018-02-2013%2015:48:10.webm
Comment 2 Cosimo Cecchi 2013-02-19 14:52:07 UTC
Yeah, I noticed this too. It's because of [1] - when we start with an extended thumbnails slider, the clip will not be updated until next time the overview is entered.
Not sure what's the best way to solve this. As far as I understand, the clip is only there to avoid the workspaces view to shine through when moving between workspaces. Perhaps the clip should always be set to the width of the primary monitor instead, and just positioned vertically.

[1] http://git.gnome.org/browse/gnome-shell/tree/js/ui/workspacesView.js#n84
Comment 3 Giovanni Campagna 2013-03-10 18:50:14 UTC
Created attachment 238538 [details] [review]
WorkspacesView: set the clip when setting the geometry

Fixes clipping the windows when you open the overview with the workspace
switcher expanded and then shrink it.

Now, not clipping horizontally as you propose might have a better performance,
but I didn't want to complicate the code to account for monitor geometry
changes. We might fix it better in 3.10.
Comment 4 Giovanni Campagna 2013-03-16 22:40:03 UTC
Hard code freeze ping!
Comment 5 Giovanni Campagna 2013-03-17 18:36:22 UTC
Attachment 238538 [details] pushed as d3ab367 - WorkspacesView: set the clip when setting the geometry
Comment 6 Giovanni Campagna 2013-03-17 18:40:39 UTC
Sorry, I pushed this by mistake.
Comment 7 drago01 2013-09-14 16:21:13 UTC
(In reply to comment #4)
> Hard code freeze ping!

(one cycle later) ... Do we still need this?
Comment 8 Giovanni Campagna 2013-09-14 16:32:03 UTC
The original reproducer doesn't work anymore, let's call this fixed.