From: Matti Tahvonen <matti.tahvonen@itmill.com>
Date: Tue, 5 Feb 2008 13:49:28 +0000 (+0000)
Subject: relocated method + added TODO
X-Git-Tag: 6.7.0.beta1~5108
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=1d9db1042c1efb5471028484efb237f3819965df;p=vaadin-framework.git

relocated method + added TODO

svn changeset:3709/svn branch:trunk
---

diff --git a/src/com/itmill/toolkit/terminal/gwt/client/ui/ITextArea.java b/src/com/itmill/toolkit/terminal/gwt/client/ui/ITextArea.java
index deb9286915..94bccd7bfd 100644
--- a/src/com/itmill/toolkit/terminal/gwt/client/ui/ITextArea.java
+++ b/src/com/itmill/toolkit/terminal/gwt/client/ui/ITextArea.java
@@ -5,12 +5,16 @@
 package com.itmill.toolkit.terminal.gwt.client.ui;
 
 import com.google.gwt.user.client.DOM;
+import com.google.gwt.user.client.Element;
 import com.itmill.toolkit.terminal.gwt.client.ApplicationConnection;
 import com.itmill.toolkit.terminal.gwt.client.UIDL;
 
 /**
  * This class represents a multiline textfield (textarea).
  * 
+ * TODO consider replacing this with a RichTextArea based implementation. IE
+ * does not support CSS height for textareas in Strict mode :-(
+ * 
  * @author IT Mill Ltd.
  * 
  */
@@ -31,4 +35,16 @@ public class ITextArea extends ITextField {
         }
     }
 
+    public void setRows(int rows) {
+        setRows(getElement(), rows);
+    }
+
+    private native void setRows(Element e, int r)
+    /*-{
+    try {
+        if(e.tagName.toLowerCase() == "textarea")
+                e.rows = r;
+    } catch (e) {}
+    }-*/;
+
 }
diff --git a/src/com/itmill/toolkit/terminal/gwt/client/ui/ITextField.java b/src/com/itmill/toolkit/terminal/gwt/client/ui/ITextField.java
index 7152d32c08..278011fec0 100644
--- a/src/com/itmill/toolkit/terminal/gwt/client/ui/ITextField.java
+++ b/src/com/itmill/toolkit/terminal/gwt/client/ui/ITextField.java
@@ -86,30 +86,21 @@ public class ITextField extends TextBoxBase implements Paintable,
         setColumns(getElement(), columns);
     }
 
-    public void setRows(int rows) {
-        setRows(getElement(), rows);
-    }
+    private native void setColumns(Element e, int c)
+    /*-{
+    try {
+    	switch(e.tagName.toLowerCase()) {
+    		case "input":
+    			//e.size = c;
+    			e.style.width = c+"em";
+    			break;
+    		case "textarea":
+    			//e.cols = c;
+    			e.style.width = c+"em";
+    			break;
+    		default:;
+    	}
+    } catch (e) {}
+    }-*/;
 
-    private native void setColumns(Element e, int c) /*-{
-                            try {
-                            	switch(e.tagName.toLowerCase()) {
-                            		case "input":
-                            			//e.size = c;
-                            			e.style.width = c+"em";
-                            			break;
-                            		case "textarea":
-                            			//e.cols = c;
-                            			e.style.width = c+"em";
-                            			break;
-                            		default:;
-                            	}
-                            } catch (e) {}
-                            }-*/;
-
-    private native void setRows(Element e, int r) /*-{
-                            try {
-                            	if(e.tagName.toLowerCase() == "textarea")
-                            		e.rows = r;
-                            } catch (e) {}
-                            }-*/;
 }