Fix compose textarea sizing in dimp

diff --git a/imp/js/compose-dimp.js b/imp/js/compose-dimp.js
index e0b9d65..19a5e42 100644 (file)
--- a/imp/js/compose-dimp.js
+++ b/imp/js/compose-dimp.js
@@ -693,10 +693,14 @@ var DimpCompose = {
              * row, and resize the textarea. */
             rows = parseInt(mah / (msg.clientHeight / msg.readAttribute('rows')), 10);
             if (!isNaN(rows)) {
-                msg.writeAttribute({ rows: rows, disabled: false });
-                if (de.scrollHeight - de.clientHeight) {
-                    msg.writeAttribute('rows', rows - 1);
-                }
+                /* Due to the funky (broken) way some browsers (FF) count
+                 * rows, we need to overshoot row estimate and increment
+                 * downward until textarea size does not cause window
+                 * scrolling. */
+                ++rows;
+                do {
+                    msg.writeAttribute({ rows: rows--, disabled: false });
+                } while (de.scrollHeight - de.clientHeight);
             }
         }
     },