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 604963 - Fix application thumbnail sizing when the height is greater than the width of a window.
Fix application thumbnail sizing when the height is greater than the width of...
Status: RESOLVED FIXED
Product: gnome-shell
Classification: Core
Component: general
unspecified
Other All
: Normal normal
: ---
Assigned To: gnome-shell-maint
gnome-shell-maint
Depends on:
Blocks:
 
 
Reported: 2009-12-19 00:04 UTC by Jasper St. Pierre (not reading bugmail)
Modified: 2010-01-05 01:21 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Fix application thumbnail sizing when the height is greater than the width of a window. (2.42 KB, patch)
2009-12-19 00:04 UTC, Jasper St. Pierre (not reading bugmail)
committed Details | Review
Fix application thumbnail sizing when the height is greater than the width of a window. (2.92 KB, patch)
2010-01-05 01:04 UTC, Jasper St. Pierre (not reading bugmail)
committed Details | Review

Description Jasper St. Pierre (not reading bugmail) 2009-12-19 00:04:07 UTC
This bug happened because the thumbnail box was only maxed, not set to 256x256.
Standard window thumbnails would max out only the width to 256. Having a window with
its height greater than its width meant that the height would max out, but not the
width, causing thumbnails to look uneven.
Comment 1 Jasper St. Pierre (not reading bugmail) 2009-12-19 00:04:09 UTC
Created attachment 150042 [details] [review]
Fix application thumbnail sizing when the height is greater than the width of a window.

This bug happened because the thumbnail box was only maxed, not set to 256x256.
Standard window thumbnails would max out only the width to 256. Having a window with
its height greater than its width meant that the height would max out, but not the
width, causing thumbnails to look uneven.
Comment 2 Colin Walters 2009-12-21 16:43:23 UTC
Review of attachment 150042 [details] [review]:

::: data/theme/gnome-shell.css
@@ +495,2 @@
 }
+.switcher-list .thumbnail {

".thumbnail" is fairly generic.  ".switcher-thumbnail"?

@@ -496,2 @@
     padding: 6px;
 .switcher-list .outlined-item-box {

I've been trying to namespace the CSS (in an admittedly ad-hoc way), and .switcher-list was there before, but...

::: js/ui/altTab.js
@@ +593,3 @@
         this._icon = this.app.create_icon_texture(POPUP_APPICON_SIZE);
                                          vertical: true });
+        this.actor.add(this._icon, { x_fill: false, y_fill: true } );

I don't understand the motivation for y_fill: true.  This means we'll be stretching the icon texture if we're allocated a larger height, which is almost always wrong.
Comment 3 Jasper St. Pierre (not reading bugmail) 2010-01-05 01:04:34 UTC
Created attachment 150809 [details] [review]
Fix application thumbnail sizing when the height is greater than the width of a window.

This bug happened because the thumbnail box was only maxed, not set to 256x256.
Standard window thumbnails would max out only the width to 256. Having a window with
its height greater than its width meant that the height would max out, but not the
width, causing thumbnails to look uneven.
Comment 4 Jasper St. Pierre (not reading bugmail) 2010-01-05 01:04:57 UTC
Well, I can rename "thumbnail" to "switcher-thumbnail" but you used "thumbnail-box" and "item-box" so I thought I would be consistent.

I removed the y_fill though.
Comment 5 Colin Walters 2010-01-05 01:21:40 UTC
Attachment 150042 [details] pushed as 9423cc7 - Fix application thumbnail sizing when the height is greater than the width of a window.
Attachment 150809 [details] pushed as 9423cc7 - Fix application thumbnail sizing when the height is greater than the width of a window.