Screenshot are displayed in full size - would it be possible to limit the height about slicerappstore HOT 9 CLOSED

This link looks better - the image is now cropped though, and it would be better to preserve the aspect ratio by only setting the height.

from slicerappstore.

That is something we could do, but I think we want each screenshot to take up the same size rectangle (180x180) so that if you have several, the thumbnails make a nice grid.

We could maintain the aspect ratio and still fit them into the grid if we use a padding approach rather than a cropping approach, but it might make for thumbnails that don't convey information very well in the case of very large aspect ratio images.

from slicerappstore.

Or are you talking about the actual full size image that appears when you click the thumbnail? If so, there is an open issue to fix that problem which I am currently working on :)

from slicerappstore.

If you want the grid to line up, I believe you could just set the width and then the hight would be padded automatically without distortion.

from slicerappstore.

If you fix the height, the aspect ratio will be preserved by making the width change. If you fix both the width and height, the browser unfortunately is not smart enough to pad by default, it just does a naive resample and distorts the image. So either way we would have to use a library like this, it's just a question of whether we prefer to crop or pad. I personally prefer the crop because it actually uses the entire alotted space, but if more users prefer to pad I will happily change it.

from slicerappstore.

did you try just fixing the width? I do prefer pad to crop.

On Mon, Apr 1, 2013 at 2:03 PM, Zach Mullen [email protected]:

If you fix the height, the aspect ratio will be preserved by making the
width change. If you fix both the width and height, the browser
unfortunately is not smart enough to pad by default, it just does a naive
resample and distorts the image. So either way we would have to use a
library like this, it's just a question of whether we prefer to crop or
pad. I personally prefer the crop because it actually uses the entire
alotted space, but if more users prefer to pad I will happily change it.

—
Reply to this email directly or view it on GitHubhttps://github.com//issues/45#issuecomment-15727596
.

from slicerappstore.

I've switched to using a pad instead of a crop, here's an example of how it looks with a wide image:

http://slicer.kitware.com/midas3/slicerappstore/extension/view?extensionId=15620&layout=empty

and with a tall image:

http://slicer.kitware.com/midas3/slicerappstore/extension/view?extensionId=15625&layout=empty

Do you like it this way or should I revert it?

from slicerappstore.

Love the padded version!

On Mon, Apr 1, 2013 at 2:26 PM, Zach Mullen [email protected]:

I've switched to using a pad instead of a crop, here's an example of how
it looks with a wide image:

http://slicer.kitware.com/midas3/slicerappstore/extension/view?extensionId=15620&layout=empty

and with a tall image:

http://slicer.kitware.com/midas3/slicerappstore/extension/view?extensionId=15625&layout=empty

Do you like it this way or should I revert it?

—
Reply to this email directly or view it on GitHubhttps://github.com//issues/45#issuecomment-15728919
.

from slicerappstore.

👍

from slicerappstore.