Attachment 206125 Details for Bug 361051 – proposed fix

Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

[patch] proposed fix

WebkitTextFix2.patch (text/plain), 2.01 KB, created by Tim Buschtoens

on 2011-10-28 08:14:50 EDT

(hide)

Description:

Filename:

MIME Type:

Creator: Tim Buschtoens

Created: 2011-10-28 08:14:50 EDT

Size: 2.01 KB

patch

obsolete

>### Eclipse Workspace Patch 1.0
>#P org.eclipse.rap.rwt
>Index: js/org/eclipse/rwt/widgets/Text.js
>===================================================================
>RCS file: /cvsroot/rt/org.eclipse.rap/runtime.rwt/org.eclipse.rap.rwt/js/org/eclipse/rwt/widgets/Text.js,v
>retrieving revision 1.4
>diff -u -r1.4 Text.js
>--- js/org/eclipse/rwt/widgets/Text.js	16 Sep 2011 12:32:24 -0000	1.4
>+++ js/org/eclipse/rwt/widgets/Text.js	28 Oct 2011 12:13:25 -0000
>@@ -52,6 +52,12 @@
>       // Fix for bug 306354
>       this._inputElement.style.paddingRight = "1px";
>     },
>+    
>+    _webkitMultilineFix : function() {
>+      if( this._inputTag !== "textarea" ) {
>+        this.base( arguments );
>+      }
>+    },
> 
>     _applyWrap : function( value, oldValue ) {
>       if( this._inputTag == "textarea" ) {
>Index: js/qx/ui/form/TextField.js
>===================================================================
>RCS file: /cvsroot/rt/org.eclipse.rap/runtime.rwt/org.eclipse.rap.rwt/js/qx/ui/form/TextField.js,v
>retrieving revision 1.5
>diff -u -r1.5 TextField.js
>--- js/qx/ui/form/TextField.js	25 Oct 2011 10:05:20 -0000	1.5
>+++ js/qx/ui/form/TextField.js	28 Oct 2011 12:13:25 -0000
>@@ -358,12 +358,27 @@
>         } else {
>           inp.addEventListener("input", this.__oninput, false);
>         }
>+        
>+        if (qx.core.Variant.isSet( "qx.client", "webkit" ) ) {
>+          this._webkitMultilineFix();
>+        }
> 
>         // TODO [tb] : write test:
>         this._getTargetNode().appendChild( inp );
>       }
>     },
>+    
>+    _webkitMultilineFix : function(){
>+      this.addEventListener( "keydown", this._preventEnter, this );
>+      this.addEventListener( "keypress", this._preventEnter, this );
>+      this.addEventListener( "keyup", this._preventEnter, this );
>+    },
> 
>+    _preventEnter : function( event ) {
>+      if( event.getKeyIdentifier() === "Enter" ) {
>+        event.preventDefault();
>+      }
>+    },
> 
>     /**
>      * We could not use width/height = 100% because the outer elements

Actions: View | Diff

Attachments on bug 361051: 206123 | 206125 | 206134