htmlarea: Fix fullscreen editor on IE6/7 -- MDL-11242

Only say px if the width is a pure number. Under IE we
get the "100%" for the width of the editor, so blindly
appending "px" results in a JS execution error.

With this patch we only attach px if it's a pure int.

Also tested the sizing of HTMLArea in the messaging
window. So both should be working now.

diff --git a/lib/editor/htmlarea/htmlarea.php b/lib/editor/htmlarea/htmlarea.php
index a7f89b033c620c5cf5e52fcc601a0145e5795745..0e682a3381c75756eca98c3207c64dbbbf2b5f14 100644 (file)
--- a/lib/editor/htmlarea/htmlarea.php
+++ b/lib/editor/htmlarea/htmlarea.php
@@ -807,12 +807,18 @@ HTMLArea.prototype.generate = function () {
     var width = (this.config.width == "auto" ? (this._toolbar.offsetWidth) : this.config.width);
     // width = Math.max(parseInt(width), 598);
 
+    width = String(width);
+    if (width.match(/^\d+$/)) { // is this a pure int? if so, let it be in px
+        width=width+"px";
+    }
+
     if (!HTMLArea.is_ie) {
         height -= 2;
         width -= 2;
     }
 
-    iframe.style.width = width + "px";
+    iframe.style.width = width;
+
     if (this.config.sizeIncludesToolbar) {
         // substract toolbar height
         height -= this._toolbar.offsetHeight;