BipadTaranMahato / slideshow

Automatically exported from code.google.com/p/slideshow
0 stars 0 forks source link

(Feature) Handle mixed-orientation sets more gracefully #25

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
Slideshow doesn't handle slideshow sets containing a mix of portrait and
landscape formatted images very well.  

I'm not entirely certain what to suggest, maybe some way to dynamically
change the size of the slideshow divs if the image orientation changes?

Original issue reported on code.google.com by jonst...@gmail.com on 6 Jul 2008 at 5:11

GoogleCodeExporter commented 9 years ago
Can you provide a URL with an example of what you think needs improvement? 
Thank you.

Original comment by aeron.gl...@gmail.com on 6 Jul 2008 at 3:13

GoogleCodeExporter commented 9 years ago
Basically what we want is a 'resize' variant that fits the image's *longest*
dimension within the frame of the slideshow, rather the current behavior (which 
fits
the *shortest* dimension and thus fills the frame entirely).

I'm attaching a patch which achieves this (though without making it 
configurable,
which you'll probably want to add).  It also modifies the transition to fade 
out the
old image even when 'overlap' is not true, which was needed to get the 
transition to
look reasonable when using a mix of image sizes.

Thanks for all your work on such a solid product.  We're almost ready to 
release the
new Plone slideshowfolder incorporating the newest slideshow.js.

Original comment by dgl...@gmail.com on 8 Jul 2008 at 2:38

Attachments:

GoogleCodeExporter commented 9 years ago
This should be incorporated now in SVN - thank you!

Original comment by aeron.gl...@gmail.com on 19 Jul 2008 at 9:21