RFR: 8224261: JProgressBar always with border painted around it [v2]

Damon Nguyen dnguyen at openjdk.org
Fri Nov 17 23:08:32 UTC 2023


On Fri, 17 Nov 2023 15:00:49 GMT, Abhishek Kumar <abhiscxk at openjdk.org> wrote:

>> JProgressBar is always painted with border irrespective of the value set via the API `setBorderPainted(boolean value)` in Synth (Nimbus and GTK) LAF. Proposed fix is to add a check before painting the component.
>> 
>> CI jobs are green after the fix. Links attached to JBS.
>
> Abhishek Kumar has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Review comment fix

Changes look logical and work as expected.

test/jdk/javax/swing/JProgressBar/TestProgressBarBorder.java line 70:

> 68:             isImgSame = Util.compareBufferedImages(borderPaintedImg, borderNotPaintedImg);
> 69: 
> 70:             if (isImgSame) {

Maybe worth renaming `isImgSame` to something that reads slightly smoother like `isImgEqual`, `isEqual`, or `isMatching`. Could just even inser the `Util.compareBufferedImages()` into the if statement for the failure check if you prefer. But I like the variable for better readability.

test/jdk/javax/swing/JProgressBar/TestProgressBarBorder.java line 92:

> 90:     private static void createAndShowUI() {
> 91:         progressBar = new JProgressBar();
> 92:         progressBar.setSize(100,50);

Suggestion:

        progressBar.setSize(100, 50);

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

Marked as reviewed by dnguyen (Committer).

PR Review: https://git.openjdk.org/jdk/pull/16467#pullrequestreview-1738070215
PR Review Comment: https://git.openjdk.org/jdk/pull/16467#discussion_r1397976092
PR Review Comment: https://git.openjdk.org/jdk/pull/16467#discussion_r1397974704


More information about the client-libs-dev mailing list