From 60a233a91772ebf8be5b66096e05d16e90f9a20f Mon Sep 17 00:00:00 2001
From: Jacques Distler <distler@golem.ph.utexas.edu>
Date: Sat, 12 Dec 2009 01:12:52 -0600
Subject: [PATCH] More Robust

Browsers have a bewildering array of
default behaviours for <textarea>s.

This will work a little more uniformly
across brosswers.
---
 public/javascripts/page_helper.js | 23 ++++++++++++++---------
 public/stylesheets/instiki.css    |  5 +++++
 2 files changed, 19 insertions(+), 9 deletions(-)

diff --git a/public/javascripts/page_helper.js b/public/javascripts/page_helper.js
index e63da41b..9115c8bd 100644
--- a/public/javascripts/page_helper.js
+++ b/public/javascripts/page_helper.js
@@ -53,14 +53,17 @@ function mactionWorkarounds() {
      });
 }
 
-function updateSize(elt) {
+function updateSize(elt, w, h) {
    // adjust to the size of the user's browser area.
+   // w and h are the original, unadjusted, width and height per row/column
     var parentheight = document.viewport.getHeight() - $('pageName').getHeight() 
-                               - $('editFormButtons').getHeight() - $('hidebutton').getHeight();
-    var parentwidth = Math.min( document.viewport.getWidth(), elt.parentNode.getWidth() ) - 10 ;
+                 - $('editFormButtons').getHeight() - $('hidebutton').getHeight();
+    var parentwidth = $('Content').getWidth();
     var f = $('MarkupHelp');
-    if (f.visible()) {parentwidth = parentwidth - f.getWidth()}
-    elt.writeAttribute({'cols': Math.floor(parentwidth/10), 'rows': Math.floor(parentheight/20)} );
+    if (f.visible()) {parentwidth = parentwidth - f.getWidth() - 20}
+    var newcols = Math.floor(parentwidth/w) - 1;
+    var newrows = Math.floor(parentheight/h - 3);
+    elt.writeAttribute({'cols': newcols, 'rows': newrows });
     elt.setStyle({Width: parentwidth, Height: parentheight});
 }
 
@@ -72,19 +75,21 @@ function resizeableTextarea() {
     f.insert({before: hidebutton});
   }
   $$('textarea#content').each( function(textarea)  {
+    var w = textarea.getWidth()/textarea.getAttribute('cols');
+    var h = textarea.getStyle('lineHeight').replace(/(\d*)px/, "$1");
     Event.observe(hidebutton, 'click', function(){
       if (f.visible()) {
         f.hide();
         hidebutton.writeAttribute({value: 'Show markup help'});
-        updateSize(textarea)
+        updateSize(textarea, w, h)
       } else {
         f.show();
         hidebutton.writeAttribute({value: 'Hide markup help'});  
-        updateSize(textarea)    
+        updateSize(textarea, w, h)    
       }
     });
-    Event.observe(window, 'resize', function(){ updateSize(textarea) });
-    updateSize(textarea);
+    Event.observe(window, 'resize', function(){ updateSize(textarea, w, h) });
+    updateSize(textarea, w, h);
    });
 }
 
diff --git a/public/stylesheets/instiki.css b/public/stylesheets/instiki.css
index d61bc960..3056e0b3 100644
--- a/public/stylesheets/instiki.css
+++ b/public/stylesheets/instiki.css
@@ -408,6 +408,11 @@ span.wikilink-error b {
  color: #009;
 }
 
+textarea#content {
+font-size: 1em;
+line-height: 1.125;
+}
+
 merror {display:inline;font-size:1em;}
 math[display=block] {overflow:auto;}
 math { white-space: nowrap }