<Swing Dev> RFR 8245921:Minor copy/paste issue in BasicScrollBarUI

Prasanta Sadhukhan prasanta.sadhukhan at oracle.com
Thu Jun 25 11:23:46 UTC 2020


Hi All,

Please review a small fix for a minor copy/paste error in 
javax.swing.plaf.basic.BasicScrollBarUI.installDefaults()

if ("large".equals(scaleKey)){
                 scrollBarWidth *= 1.15;
                 incrGap *= 1.15;
                 decrGap *= 1.15;
             } else if ("small".equals(scaleKey)){
                 scrollBarWidth *= 0.857;
                 incrGap *= 0.857;
                 decrGap *= 0.714;
             } else if ("mini".equals(scaleKey)){
                 scrollBarWidth *= 0.714;
                 incrGap *= 0.714;
                 decrGap *= 0.714;
             }

In the "small" case, decrGap is multiplied by 0.714 instead of 0.857.

This code snippet has been added as part of the enhancement to support 
Nimbus L&F in JDK-6591875. 
<https://bugs.openjdk.java.net/browse/JDK-6591875>

In SynthScrollBarUI.java which is also added as part of above JBS, the 
correct value is used 
http://hg.openjdk.java.net/jdk/client/annotate/43e06bc950ec/jdk/src/share/classes/javax/swing/plaf/synth/SynthScrollBarUI.java#l116

Bug: https://bugs.openjdk.java.net/browse/JDK-8245921

diff -r b1bd79b486bc 
src/java.desktop/share/classes/javax/swing/plaf/basic/BasicScrollBarUI.java
--- 
a/src/java.desktop/share/classes/javax/swing/plaf/basic/BasicScrollBarUI.java 
Wed Jun 10 12:43:22 2020 +0530
+++ 
b/src/java.desktop/share/classes/javax/swing/plaf/basic/BasicScrollBarUI.java 
Thu Jun 25 12:57:37 2020 +0530
@@ -270,7 +270,7 @@
              } else if ("small".equals(scaleKey)){
                  scrollBarWidth *= 0.857;
                  incrGap *= 0.857;
-                decrGap *= 0.714;
+                decrGap *= 0.857;
              } else if ("mini".equals(scaleKey)){
                  scrollBarWidth *= 0.714;
                  incrGap *= 0.714;

Regards
Prasanta
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.openjdk.java.net/pipermail/swing-dev/attachments/20200625/9d281acf/attachment.htm>


More information about the swing-dev mailing list