RFR: 8190264: JScrollBar ignores its border when using macOS Mac OS X Aqua look and feel [v12]
Alexey Ivanov
aivanov at openjdk.java.net
Tue Jan 11 19:47:49 UTC 2022
On Mon, 10 Jan 2022 16:59:28 GMT, Alisen Chung <achung at openjdk.org> wrote:
>> Adjusted the AquaLF scrollbar to account for border inset settings when dragging the thumb and clicking on the track.
>
> Alisen Chung 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 16 additional commits since the last revision:
>
> - Merge branch 'master' of github.com:alisenchung/jdk into 8190264
> - updated test
> - added check for other L&Fs, added newline to test
> - removed todo comment
> - finished test
> - fixed test 2
> - fixed test
> - need to fix comparison of BufferedImages
> - removed robot mouse moves, use pixel colors to check
> - removed manual tag
> - ... and 6 more: https://git.openjdk.java.net/jdk/compare/f166bd7b...ee57b8ac
Please update the copyright year in both files to include 2022.
I suggest expanding all the wildcard imports for AquaScrollBarUI.java, which is usually done.
And most importantly: the new code handle horizontal insets. What about the *vertical insets?* It looks a vertical scroll bar has the same issue if there's vertical border.
test/jdk/java/awt/Scrollbar/AquaLFScrollbarTest/ScrollBarBorderTest.java line 2:
> 1: /*
> 2: * Copyright (c) 2021, Oracle and/or its affiliates. All rights reserved.
Probably, the copyright year should now contain 2022 too:
Suggestion:
* Copyright (c) 2021, 2022, Oracle and/or its affiliates. All rights reserved.
-------------
Changes requested by aivanov (Reviewer).
PR: https://git.openjdk.java.net/jdk/pull/6374
More information about the client-libs-dev
mailing list