<Swing Dev> RFR: 8262981: Create implementation for NSAccessibilitySlider protocol [v2]
Pankaj Bansal
pbansal at openjdk.java.net
Sun Mar 14 11:18:25 UTC 2021
> Create implementation of NSAccessibilitySlider protocol https://developer.apple.com/documentation/appkit/nsaccessibilityslider
>
> The implementation has the function performIncrement and performDecrement to increase/decrease the value of slider using the VoiceOver. To implement this functionality, I could think of following two ways. I have chosen the first one here though it is more intrusive.
> 1. Make the AccessibleJSlider class implement the AccessibleAction interface. This makes AccessibleJSlider consistent with the AccessibleJSpinner class. It is more clear and the logic to increase/decrease Slider value can be changed easily. But this changes AccessibleJSlider class and will need a CSR.
> 2. Get the current Accessible Value from the component and just increment/decrement it in SliderAccessibility.m class itself and then set the current accessible value fro there only. This will not need any changes in AccessibleJSlider class, but this does not look correct way and I have not used this.
>
> The changes can be easily tested by using a JSlider example, like the following example. VO should announce the correct the slider values. To change the slider values, use ctrl+opt+shift+down key to start interacting with the slider, then use ctrl+opt+up/down arrow keys to increment/decrement slider values.
>
> import javax.swing.JFrame;
> import javax.swing.JSlider;
>
> public class JSliderDemo{
> public static void main(String[] args) throws Exception {
> JFrame jframe = new JFrame("JSlider Demo");
> jframe.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
> jframe.add(new JSlider());
>
> jframe.setLocationRelativeTo(null);
> jframe.pack();
> jframe.setVisible(true);
> }
> }
Pankaj Bansal has updated the pull request incrementally with one additional commit since the last revision:
increase initial capacity for rolesMap
-------------
Changes:
- all: https://git.openjdk.java.net/jdk/pull/2874/files
- new: https://git.openjdk.java.net/jdk/pull/2874/files/dac91ff1..a1944fee
Webrevs:
- full: https://webrevs.openjdk.java.net/?repo=jdk&pr=2874&range=01
- incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=2874&range=00-01
Stats: 1 line in 1 file changed: 0 ins; 0 del; 1 mod
Patch: https://git.openjdk.java.net/jdk/pull/2874.diff
Fetch: git fetch https://git.openjdk.java.net/jdk pull/2874/head:pull/2874
PR: https://git.openjdk.java.net/jdk/pull/2874
More information about the swing-dev
mailing list