johnreva / jquery-lightbox

Automatically exported from code.google.com/p/jquery-lightbox
0 stars 0 forks source link

Enhancement : auto-resize images (or not) #7

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
Hi, there !

Thanks for this great plugin ! I wrote a small modification to auto-resize
large images to a certain maxwidth, maxheight. Small images can be enlarged
to these max values or left untouched.

Here is the code attached, meant to be inserted at the very beginning of
resizeImageContainer (after "resizeImageContainer: function( imgWidth,
imgHeight) {" and before "// get curren width and height").

Cheers,
Sylvain

Original issue reported on code.google.com by sylvain....@gmail.com on 14 Mar 2008 at 4:17

Attachments:

GoogleCodeExporter commented 8 years ago
i think its good thing, i miss this function in all lightbox implementations

Original comment by ghi...@gmail.com on 6 May 2008 at 4:55

GoogleCodeExporter commented 8 years ago
We do have something like this in Version 0.5.  I'll take a look and see which
implementation fits better with the project.  In either case, thanks for the 
patch!

Original comment by krewenki on 21 May 2008 at 1:29

GoogleCodeExporter commented 8 years ago
I borrowed resize part from fancybox and it works great! it calc width and 
height
from window's dimensions, really better!
fancybx is under MIT license would be correct to write this code here?

Original comment by tbarbu...@gmail.com on 21 Jan 2009 at 11:08

GoogleCodeExporter commented 8 years ago
There have been multiple changes made to this project since it's move to 
github, several months ago.  You can 
find the project at http://github.com/krewenki/jquery-lightbox/tree/master .   
Please grab the latest copy from 
github, and if your issue persists, create a new issue there.

Original comment by krewenki on 17 Jun 2009 at 5:03

GoogleCodeExporter commented 8 years ago
Great work.Thanks.

Original comment by sami.md...@gmail.com on 26 Apr 2012 at 2:37