Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
View | Details | Raw Unified | Return to bug 343820
Collapse All | Expand All

(-)js/org/eclipse/swt/widgets/AbstractSlider.js (-5 / +9 lines)
Lines 171-177 Link Here
171
    },
171
    },
172
172
173
    _onMouseOut : function( event ) {
173
    _onMouseOut : function( event ) {
174
      if( this._autoRepeat.slice( 0, 4 ) === "line" ) {
174
      var target = event.getRelatedTarget();
175
      var outOfSlider = target !== this && !this.contains( target );
176
      if( outOfSlider && this._autoRepeat.slice( 0, 4 ) === "line" ) {
175
        this.setCapture( true );
177
        this.setCapture( true );
176
        this._delayTimer.stop();
178
        this._delayTimer.stop();
177
        this._repeatTimer.stop();
179
        this._repeatTimer.stop();
Lines 223-232 Link Here
223
    },
225
    },
224
226
225
    _onThumbMouseUp : function( event ) {
227
    _onThumbMouseUp : function( event ) {
226
      event.stopPropagation();
228
      if( this._thumb.hasState( "pressed" ) ) {
227
      this._repeatTimer.stop();
229
        event.stopPropagation();
228
      this._thumb.setCapture( false );
230
        this._repeatTimer.stop();
229
      this._thumb.removeState( "pressed" );
231
        this._thumb.setCapture( false );
232
        this._thumb.removeState( "pressed" );
233
      }
230
    },
234
    },
231
235
232
    ////////////
236
    ////////////
(-)js/org/eclipse/rwt/test/tests/SliderTest.js (+18 lines)
Lines 443-448 Link Here
443
      slider.destroy();
443
      slider.destroy();
444
    },
444
    },
445
445
446
    testHoldOnLineMouseUpOnThumb : function() {
447
      var testUtil = org.eclipse.rwt.test.fixture.TestUtil;
448
      var slider = this._createSlider( false );
449
      var node = slider.getElement();
450
      var left = qx.event.type.MouseEvent.buttons.left;
451
      var thumb = slider._thumb.getElement();
452
      testUtil.fakeMouseEventDOM( node, "mousedown", left, 11, 50 );
453
      assertEquals( 10, slider._selection );
454
      testUtil.fakeMouseEventDOM( thumb, "mouseup", left, 11, 50 );
455
      try {
456
        testUtil.forceInterval( slider._delayTimer ); // start scrolling
457
      } catch( ex ) {
458
        // expected
459
      }
460
      assertFalse( slider._repeatTimer.isEnabled() );
461
      slider.destroy();
462
    },
463
446
    testHoldOnLineMouseOut : function() {
464
    testHoldOnLineMouseOut : function() {
447
      var testUtil = org.eclipse.rwt.test.fixture.TestUtil;
465
      var testUtil = org.eclipse.rwt.test.fixture.TestUtil;
448
      var slider = this._createSlider( false );
466
      var slider = this._createSlider( false );

Return to bug 343820