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 694210 - window thumbnails: drop grid alignment
window thumbnails: drop grid alignment
Status: RESOLVED FIXED
Product: gnome-shell
Classification: Core
Component: overview
3.7.x
Other Linux
: Normal normal
: ---
Assigned To: gnome-shell-maint
gnome-shell-maint
Depends on:
Blocks:
 
 
Reported: 2013-02-19 20:13 UTC by Allan Day
Modified: 2013-02-19 23:44 UTC
See Also:
GNOME target: ---
GNOME version: 3.7/3.8


Attachments
patch (1021 bytes, text/plain)
2013-02-19 20:13 UTC, Allan Day
  Details
workspace: Don't use grid layout (3.50 KB, patch)
2013-02-19 22:14 UTC, drago01
committed Details | Review

Description Allan Day 2013-02-19 20:13:00 UTC
Right now we arrange the window thumbnails in a grid if there are more than two rows of them. This was originally intended to reduce the amount of noise in the thumbnail arrangement. It also made sense when the thumbnails were of a similar size.

Nowadays we reflect the size of windows in the size of the thumbnail. This leads to huge amounts of space between some windows when they are grid aligned. Removing the grid alignment would also lead to larger thumbnails.

I think we should try and drop the grid layout before 3.8.
Comment 1 Allan Day 2013-02-19 20:13:34 UTC
Created attachment 236829 [details]
patch

Here's an experimental patch from Jasper. We can use it for testing.
Comment 2 drago01 2013-02-19 22:14:58 UTC
Created attachment 236845 [details] [review]
workspace: Don't use grid layout

Right now we arrange the window thumbnails in a grid if there are more than two
rows of them. This was originally intended to reduce the amount of noise in the
thumbnail arrangement. It also made sense when the thumbnails were of a similar
size.

Nowadays we reflect the size of windows in the size of the thumbnail. This
leads to huge amounts of space between some windows when they are grid aligned.
Removing the grid alignment would also lead to larger thumbnails.
Comment 3 Jasper St. Pierre (not reading bugmail) 2013-02-19 22:22:33 UTC
Review of attachment 236845 [details] [review]:

Still needs a designer ACK, but this is more like the patch that I would have written. There's still leftover cruft (LayoutStrategy base class, extra things that can be calculated) of course, but that's follow-up stuff.

Marking Reviewed -- treat as ACN after the design team says they want it, and I'll write some follow-up patches after 3.8 release.
Comment 4 Jakub Steiner 2013-02-19 23:19:19 UTC
I am for this change. The varying aspect ratio of the window thumbnails makes the grid alignment matter of luck and it seems to just create random amounts of whitespace. I am for a more compact view for the window arrangement.
Comment 5 drago01 2013-02-19 23:44:51 UTC
Attachment 236845 [details] pushed as 4b3c9e8 - workspace: Don't use grid layout