Bugfix maxHeight after $.colorbox.resize()

Earlier, if I set .colorbox{maxHeight:'95%'...} then colorbox frame gets vertical scrollbar (if content is heighter than frame) but if then I do $.colorbox.resize() my colorbox frame becomes heighter than wrapper page and my maxHeight option is ignored, so this is bugfix for it.
Maybe then settings.h = Math.min($loaded.height(), settings.mh); ?


2014-08-13 11:55 GMT+03:00 Alexander Obuhovich <notifications@github.com>:

> In jquery.colorbox.js:
>
> > @@ -733,6 +733,7 @@
> >  				scrolltop = $loaded.scrollTop();
> >  				$loaded.css({height: "auto"});
> >  				settings.h = $loaded.height();
> > +				if (settings.h > settings.mh) settings.h=settings.mh;
>
> Great, please make necessary adjustments to your PR code then. Also don't
> forget to squash the commits.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/jackmoore/colorbox/pull/635/files#r16164162>.
>