Imagebox corrections proposal

[Spyroland]

Here are some change proposals for imagebox:

490,494d489 < center the outercontainer, so you can click on the left < or the right to close the imagebox < outerContainer < .css('width', containerW + 'px') < .css('left', (clientSize.w-containerW)/2 + 'px');

566,573d560 < < if the page does not occupy the whole window, < and the image is taller than the window, < the overlay needs to be extended now (= after resizing the image box itself) < (otherwhise scrolling reveals the original background) < overlay = jQuery('#ImageBoxOverlay'); < pageSize = jQuery.iUtil.getScroll(); < overlay.css ('height', Math.max(pageSize.ih,pageSize.h) + 'px')

[Spyroland]

Ooops sorry for the formatting

490,494d489
<	// center the outercontainer, so you can click on the left
<	// or the right to close the imagebox
<	outerContainer
<	.css('width', containerW + 'px')
<	.css('left', (clientSize.w-containerW)/2 + 'px');


566,573d560
<	////
<	// if the page does not occupy the whole window,
< 	// and the image is taller than the window,
< 	// the overlay needs to be extended now (= after resizing the image box itself)
< 	// (otherwhise scrolling reveals the original background)
< 	overlay = jQuery('#ImageBoxOverlay');
< 	pageSize = jQuery.iUtil.getScroll();
< 	overlay.css ('height', Math.max(pageSize.ih,pageSize.h) + 'px')

[dmethvin]

This is not a jQuery core bug. Please report plugin bugs to the plugin's author, or ask on the jQuery forums. jQuery UI bugs should be reported on the UI bug tracker, ​http://dev.jqueryui.com .