On Tue, 29 Aug 2023 20:14:44 GMT, Alexander Zuev <kiz...@openjdk.org> wrote:
>> Create implementation for Slider and Stepper accessibility protocols. >> Fix mapping. >> Fix performAction parameter type in declaration. > > Alexander Zuev has updated the pull request incrementally with one additional > commit since the last revision: > > Add commented out debug statements. I observed few warnings and behavioral differences. - Warnings: 1. modules/javafx.graphics/src/main/native-glass/mac/a11y/JFXSliderAccessibility.m:32:17: warning: method 'accessibilityFrame' in protocol 'NSAccessibilityElement' not implemented [-Wprotocol] 2. modules/javafx.graphics/src/main/native-glass/mac/a11y/JFXSliderAccessibility.m:32:17: warning: method 'accessibilityParent' in protocol 'NSAccessibilityElement' not implemented [-Wprotocol] 3. modules/javafx.graphics/src/main/native-glass/mac/a11y/JFXStepperAccessibility.m:32:17: warning: method 'accessibilityFrame' in protocol 'NSAccessibilityElement' not implemented [-Wprotocol] 4. modules/javafx.graphics/src/main/native-glass/mac/a11y/JFXStepperAccessibility.m:32:17: warning: method 'accessibilityParent' in protocol 'NSAccessibilityElement' not implemented [-Wprotocol] 5. modules/javafx.graphics/src/main/native-glass/mac/a11y/AccessibleBase.m:247:21: warning: 'NSObject' may not respond to 'clearParent' - Behavior differences: 1. Spinner: Changed values are not read by VoiceOver when we increase or decrease the value using arrow keys or using Ctrl + Option + Up/Down keys ( Tested with Spinner app in Ensemble ) 2. Slider: Values are read without % as suffix ( Tested with Styled tool bar app in Ensemble ) ------------- PR Comment: https://git.openjdk.org/jfx/pull/1226#issuecomment-1699283284