> Issue is
> BasicScrollBarUI.ArrowButtonListener starts a timer in mousePressed(), and 
> stops it in mouseReleased(). If the frame containing the scrollbar is 
> disabled between the MOUSE_PRESSED and the MOUSE_RELEASED events, the 
> mouseReleased() method is never called. If the frame is then re-enabled, the 
> still-running timer causes it to scroll all the way to the end.
> Fix is to check if 
> [ArrowButtonListener.handledEvent](https://github.com/openjdk/jdk/blame/ee839b7f0ebe471d3877cddd2c87019ccb8ee5ae/src/java.desktop/share/classes/javax/swing/plaf/basic/BasicScrollBarUI.java#L1567)
>  is still set when ActionEvent is processed then stop the timer and reset 
> this variable.
> 
> CI testing is green and also SwingSet2 JScrollPane scrolling with this 
> modification..

Prasanta Sadhukhan has updated the pull request incrementally with two 
additional commits since the last revision:

 - copyright
 - Restore AquaL&F testing

-------------

Changes:
  - all: https://git.openjdk.org/jdk/pull/20346/files
  - new: https://git.openjdk.org/jdk/pull/20346/files/53d6f32e..00fd5e58

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=20346&range=02
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=20346&range=01-02

  Stats: 11 lines in 3 files changed: 6 ins; 3 del; 2 mod
  Patch: https://git.openjdk.org/jdk/pull/20346.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/20346/head:pull/20346

PR: https://git.openjdk.org/jdk/pull/20346

Reply via email to