Hi All,
I am using blockUI(2.14) in one of my projects.
A few days back, I upgraded to IE8 and the site stopped working. On
debugging, I found the issue to be in blockUI plugin.
Apparently, the reason seems to be that the plugin uses dynamic
functions like setExpression which are not supported in IE8 (I am
using 8.0.6001.18702).
I have generated the patch for the same. I could not find a way to
submit the patch for verification. So, I am adding it here.
Please let me know if this works fine.
Regards,
Viraj
225,230c225,236
< full ? s.setExpression('height','Math.max
(document.body.scrollHeight, document.body.offsetHeight) -
(jQuery.boxModel?0:'+opts.quirksmodeOffsetHack+') + "px"')
< : s.setExpression
('height','this.parentNode.offsetHeight + "px"');
< full ? s.setExpression('width','jQuery.boxModel &&
document.documentElement.clientWidth || document.body.clientWidth +
"px"')
< : s.setExpression
('width','this.parentNode.offsetWidth + "px"');
< if (fixL) s.setExpression('left', fixL);
< if (fixT) s.setExpression('top', fixT);
---
> if(full) {
> s.height = Math.max(document.body.scrollHeight,
> document.body.offsetHeight) - (jQuery.boxModel ? 0:
> opts.quirksmodeOffsetHack) + "px";
> } else {
> s.height = this.parentNode.offsetHeight + "px";
> }
> if(full) {
> s.width = jQuery.boxModel &&
> document.documentElement.clientWidth || document.body.clientWidth + "px";
> } else {
> s.width = this.parentNode.offsetWidth + "px";
> }
> if (fixL) s.left = fixL;
> if (fixT) s.top = fixT;
233c239
< if (full) s.setExpression
('top','(document.documentElement.clientHeight ||
document.body.clientHeight) / 2 - (this.offsetHeight / 2) + (blah =
document.documentElement.scrollTop ?
document.documentElement.scrollTop : document.body.scrollTop) +
"px"');
---
> if (full) s.top = (document.documentElement.clientHeight ||
> document.body.clientHeight) / 2 - (this[0].offsetHeight / 2) + (blah =
> document.documentElement.scrollTop ? document.documentElement.scrollTop :
> document.body.scrollTop) + "px";
236,240c242,245
< else if (!opts.centerY && full) {
< var top = (opts.css && opts.css.top) ?
parseInt(opts.css.top) :
0;
< var expression =
'((document.documentElement.scrollTop ?
document.documentElement.scrollTop : document.body.scrollTop) + '+top
+') + "px"';
< s.setExpression('top',expression);
< }
---
> else if (!opts.centerY && full) {
> var top = (opts.css && opts.css.top) ? parseInt(opts.css.top) :
> 0;
> s.top = ((document.documentElement.scrollTop ?
> document.documentElement.scrollTop : document.body.scrollTop) + top) + "px";
> }
