<Swing Dev> Review Request: JDK-8009477 :PageUp and PageDown keyboard buttons don't move slider indicator to next minor tick

[Prem Balakrishnan]

Hi Alexandr,

 

Thankyou for the Review.

 

>  For the case there only majorTickSpacing is set should it be used for a block increment?

When setSnapToTicks(Boolean value) is set to true, 

All the keys Left/Right, Up/Down, PageUp/PageDown snaps to nearest tick(i.e., If minor Tick not set, it considers major Tick as the nearest tick).

 

Based on my understanding, we need to set the  minimum blockincrement size for PageUp/PageDown, which should be MinorTickSpacing if available.

Where as if minorTickSpacing not Set and only MajorTickSpacing is set then blockincrement=[ (slider.getMaximum() - slider.getMinimum()) / 10] should be better.

 

Regards,

Prem

 

From: Alexandr Scherbatiy 
Sent: Thursday, September 01, 2016 8:46 PM
To: Prem Balakrishnan; Sergey Bylokhov; Rajeev Chamyal; swing-dev at openjdk.java.net
Subject: Re: Review Request: JDK-8009477 :PageUp and PageDown keyboard buttons don't move slider indicator to next minor tick

 

On 8/31/2016 3:36 PM, Prem Balakrishnan wrote:



Hi,

Please review fix for JDK 9,

 

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

Webrev: HYPERLINK "http://cr.openjdk.java.net/%7Epkbalakr/8009477/webrev.00/"http://cr.openjdk.java.net/~pkbalakr/8009477/webrev.00/ 

 

Cause: PageUp/PageDown key is used to scroll by block, 

            MinorTickSpacing set by the user is not consider as the Block size.

 

Fix:    MinorTickSpacing(non-zero value) set by the user is set as Block size.

  For the case there only majorTickSpacing is set should it be used for a block increment?

  Thanks,
  Alexandr.



            

Regards,

Prem

 
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/swing-dev/attachments/20160901/f49e1e37/attachment.html>