RFR: 6672644: JComboBox still scrolling if switch to another window and return back [v7]
Damon Nguyen
dnguyen at openjdk.org
Thu Nov 21 16:51:12 UTC 2024
> In a JComboBox, if the user opens the dropdown list and clicks and holds the down-button, then ALT-TABs to switch focus, when the user re-focuses the frame with the JComboBox and opens the dropdown list again, the list will still be scrolling even though the down-button isn't pressed.
>
> This isn't OS or L&F specific, although Aqua L&F does not have any directional arrows in the dropdown list (and is thus exempt). This led me to believe it could be handled in BasicComboBoxUI where focusLost and focusGain are used or isPopupVisible but the scroll behavior cannot be altered here. Likewise for BasicComboPopup where `autoscroll` is used. However, this behavior isn't related to autoscroll and is actually found in the JScrollbar of the JScrollpane inside of the JComboBox. The timer for the scroll action starts but is never stopped if focus is lost, so a new listener is created and used. The proposed solution uses `KeyboardFocusManager` to track the focus owner. The listener stops the `scrollTimer` when the `focusOwner` property is changed. With this change, the list no longer automatically scrolls when re-focused and instead opens normally.
>
> The included test is manual due to the need to confirm that the list still scrolls after ALT-TABing. The L&F is set to Metal since it is the cross-platform lookandfeel and has directional buttons for the JScrollPane list.
Damon Nguyen has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains 10 additional commits since the last revision:
- Review comments
- Merge branch 'master' into 6672644/focusScroll
- isPressed check and set
- Review comments
- Add space
- Update fix to resolve button pressed appearance. Handle review comments
- Merge branch 'master' into 6672644/focusScroll
- Initial commit
- Review comments
- Initial commit
-------------
Changes:
- all: https://git.openjdk.org/jdk/pull/20845/files
- new: https://git.openjdk.org/jdk/pull/20845/files/2f6566c5..0b1ae7a1
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jdk&pr=20845&range=06
- incr: https://webrevs.openjdk.org/?repo=jdk&pr=20845&range=05-06
Stats: 432734 lines in 5943 files changed: 213324 ins; 181586 del; 37824 mod
Patch: https://git.openjdk.org/jdk/pull/20845.diff
Fetch: git fetch https://git.openjdk.org/jdk.git pull/20845/head:pull/20845
PR: https://git.openjdk.org/jdk/pull/20845
More information about the client-libs-dev
mailing list