From 5451e3ae55df37d2375e75c9d6fe46f935941aab Mon Sep 17 00:00:00 2001
From: "stefanh%inbox.com"
 <stefanh%inbox.com@18797224-902f-48f8-a5cc-f745e15eee43>
Date: Sun, 25 May 2008 19:16:57 +0000
Subject: [PATCH] Bug 432132 - Use <textbox type=number> in Composer where
 applicable (preference part). Also removed one obsolete function. NPOTDB.
 r+sr=Neil.

git-svn-id: svn://10.0.0.236/trunk@251814 18797224-902f-48f8-a5cc-f745e15eee43
---
 mozilla/editor/ui/composer/content/pref-composer.xul | 12 +++++++-----
 .../editor/ui/dialogs/content/EdAEHTMLAttributes.js  |  4 ++--
 mozilla/editor/ui/dialogs/content/EdDialogCommon.js  | 11 -----------
 3 files changed, 9 insertions(+), 18 deletions(-)

diff --git a/mozilla/editor/ui/composer/content/pref-composer.xul b/mozilla/editor/ui/composer/content/pref-composer.xul
index fac9a8b605f..d1680e9ebef 100644
--- a/mozilla/editor/ui/composer/content/pref-composer.xul
+++ b/mozilla/editor/ui/composer/content/pref-composer.xul
@@ -80,11 +80,13 @@
                accesskey="&documentsInMenu.accesskey;"
                control="recentFiles"/>
         <textbox id="recentFiles"
-                 name="recent string" 
-                 size="3" 
-                 value="10" 
-                 preference="editor.history.url_maximum" 
-                 oninput=" ValidateNumber(this, null, 0, 99); LimitStringLength('recentFiles',2);"/>
+                 name="recent string"
+                 type="number"
+                 max="99"
+                 min="0"
+                 size="3"
+                 value="10"
+                 preference="editor.history.url_maximum"/>
       </hbox>
     </groupbox>
 
diff --git a/mozilla/editor/ui/dialogs/content/EdAEHTMLAttributes.js b/mozilla/editor/ui/dialogs/content/EdAEHTMLAttributes.js
index a56ff364178..e93bef4a013 100644
--- a/mozilla/editor/ui/dialogs/content/EdAEHTMLAttributes.js
+++ b/mozilla/editor/ui/dialogs/content/EdAEHTMLAttributes.js
@@ -303,8 +303,8 @@ function onInputHTMLAttributeValue()
   if (value)
   {
     // Do value filtering based on type of attribute
-    // (Do not use "LimitStringLength()" and "forceInteger()"
-    //  to avoid multiple reseting of input's value and flickering)
+    // (Do not use "forceInteger()" to avoid multiple
+    //  input's value and flickering)
     var selectedItem = gDialog.AddHTMLAttributeNameInput.selectedItem;
 
     if (selectedItem)
diff --git a/mozilla/editor/ui/dialogs/content/EdDialogCommon.js b/mozilla/editor/ui/dialogs/content/EdDialogCommon.js
index 936b9706d52..f1d91acdd9e 100644
--- a/mozilla/editor/ui/dialogs/content/EdDialogCommon.js
+++ b/mozilla/editor/ui/dialogs/content/EdDialogCommon.js
@@ -273,17 +273,6 @@ function forceInteger(elementID)
   }
 }
 
-function LimitStringLength(elementID, length)
-{
-  var editField = document.getElementById( elementID );
-  if ( !editField )
-    return;
-
-  var stringIn = editField.value;
-  if (stringIn && stringIn.length > length)
-    editField.value = stringIn.slice(0,length);
-}
-
 function InitPixelOrPercentMenulist(elementForAtt, elementInDoc, attribute, menulistID, defaultIndex)
 {
   if (!defaultIndex) defaultIndex = gPixel;