In Drupal 7, screenshot.png is 294px wide. So that is the standard size you create it in. But this is how admin/appearance looks in Rubik: http://screencast.com/t/93uDEzey4

It kinda works with most themes, as they show enough detail in the remaining upper left fourth of the image. But with my theme Screenshot (even if not pretty) it became obvious, that this should be handled differently.
If Rubik does not display the big thumbnails, it should simply resize them so they are displayed in small size correctly.
I guess simple resizing by HTML is fine for that page, since it is an admin page.

So this would do:

.theme-selector img {
    clip: rect(0px, 120px, 80px, 0px);
    height: auto;
    left: 10px;
    position: absolute;
    top: 10px;
    width: 120px;
}

I may make a patch, but a fix should be easy enough anyway.
Giving it a bit of a border (say 1px #eee) would cater for Screenshots that are mostly white, but I did not fully succeed because of the clip property that clips before the border can come in at the bottom:
http://screencast.com/t/HTvVfCQRrZb

Comments

haydeniv’s picture

Issue summary:View changes
Status:Active» Fixed

Committed 24a4b88

I went ahead and set the screenshots to be in line with Seven's theme screenshot size and did smaller sizes for disabled themes for distinction. I also update the Rubik screenshot so it fits the space better now.

Thanks!

Status:Fixed» Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.