Attachment 181493 Details for Bug 328463 – proposed patch for HEAD

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

[patch] proposed patch for HEAD

328463_tablecolumn_HEAD.patch (text/plain), 3.20 KB, created by Istvan Ballok

on 2010-10-22 08:34:05 EDT

(hide)

Description:

Filename:

MIME Type:

Creator: Istvan Ballok

Created: 2010-10-22 08:34:05 EDT

Size: 3.20 KB

patch

obsolete

>commit 0a95f9b9195446a0ba55931debb4b6ef5642b13c
>tree c131100adc4c04dea9c087bfd4531479039b792f
>parent 18a203a06001b7ccfebb9439a823ab5aed5c4fc7
>author Istvan Ballok <Istvan.Ballok@cas.de> 1287750727 +0200
>committer Istvan Ballok <Istvan.Ballok@cas.de> 1287750727 +0200
>
>    bug 328463: [TableColumn Header] moveable columns lose and gain :hover state on single click
>    
>    The problem
>    ===========
>    Moveable columns lose and gain :hover state on single click - if the theme
>    defines different background for the default and :hover states,
>    this issue causes the TableColumn to blink on click.
>    You can easily reproduce this behaviour in the controls demo
>    with business design.
>    
>    Proposed resolution
>    ===================
>    The :hover state should be removed from the TableColumn in javascript
>    only on the first mouse move event, after the mouse down event.
>
>diff --git a/runtime.rwt/org.eclipse.rap.rwt.q07/js/org/eclipse/swt/widgets/TableColumn.js b/runtime.rwt/org.eclipse.rap.rwt.q07/js/org/eclipse/swt/widgets/TableColumn.js
>index 057ba0d..3d838f1 100644
>--- a/runtime.rwt/org.eclipse.rap.rwt.q07/js/org/eclipse/swt/widgets/TableColumn.js
>+++ b/runtime.rwt/org.eclipse.rap.rwt.q07/js/org/eclipse/swt/widgets/TableColumn.js
>@@ -64,6 +64,7 @@ qx.Class.define( "org.eclipse.swt.widgets.TableColumn", {
>       this._sortImage.setAppearance( "table-column-sort-indicator" );
>     }
>     this.add( this._sortImage );
>+    this.__performFakeMouseOutOnMouseMove = false;
>   },
> 
>   destruct : function() {
>@@ -154,7 +155,7 @@ qx.Class.define( "org.eclipse.swt.widgets.TableColumn", {
>           this.setCapture( true );
>           evt.stopPropagation();
>           evt.preventDefault();
>-          widgetUtil._fakeMouseEvent( this, "mouseout" );
>+          this.__performFakeMouseOutOnMouseMove = true;
>         } else if( this._moveable ) {
>           this._inMove = true;
>           this.setCapture( true );
>@@ -165,7 +166,7 @@ qx.Class.define( "org.eclipse.swt.widgets.TableColumn", {
>           this._initialLeft = this.getLeft();
>           evt.stopPropagation();
>           evt.preventDefault();
>-          widgetUtil._fakeMouseEvent( this, "mouseout" );
>+          this.__performFakeMouseOutOnMouseMove = true;
>         }
>       }
>     },
>@@ -207,6 +208,10 @@ qx.Class.define( "org.eclipse.swt.widgets.TableColumn", {
> 
>     _onMouseMove : function( evt ) {
>       if( this._inResize ) {
>+        if (this.__performFakeMouseOutOnMouseMove) {
>+          this.__performFakeMouseOutOnMouseMove = false;
>+          widgetUtil._fakeMouseEvent( this, "mouseout" );
>+        }
>         var position = this.getLeft() + this._getResizeWidth( evt.getPageX() );
>         // min column width is 5 px
>         if( position < this.getLeft() + 5 ) {
>@@ -214,6 +219,10 @@ qx.Class.define( "org.eclipse.swt.widgets.TableColumn", {
>         }
>         this._table._showResizeLine( position );
>       } else if( this._inMove ) {
>+        if (this.__performFakeMouseOutOnMouseMove) {
>+          this.__performFakeMouseOutOnMouseMove = false;
>+          widgetUtil._fakeMouseEvent( this, "mouseout" );
>+        }
>         this.setLeft( evt.getPageX() - this._offsetX );
>         this.addState( org.eclipse.swt.widgets.TableColumn.STATE_MOVING );
>       } else {

Actions: View | Diff

Attachments on bug 328463: 181493 | 182632