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

[Alexandr Scherbatiy]

On 9/2/2016 8:28 AM, Prem Balakrishnan wrote:
>
> 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.
>
   I see that the minorTickSpacing and majorTickSpacing are already used 
for the block increment in the BasicSliderUI.getTickSpacing() method 
when slider.getSnapToTicks() is enabled. May be a user just needs to set 
snapToTicks to true for a slider to get the desired block increment?

   Thanks,
   Alexandr.

> 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:*http://cr.openjdk.java.net/~pkbalakr/8009477/webrev.00/ 
> <http://cr.openjdk.java.net/%7Epkbalakr/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/20160902/0c1e62f3/attachment.html>