<Swing Dev> RfR JDK-8076554, [macosx] Custom Swing text components need to allow standard accessibility

Phil Race philip.race at oracle.com
Tue Apr 19 17:21:40 UTC 2016


Which seems to imply the _AccessibleState subclass has been obsolete 
since 1.5
Although when these tools were unbundled it perhaps could not assume 1.5.

The rest of the changes look fine and it is a lot simpler than the v0 ..

-phil.

On 04/19/2016 08:54 AM, Pete Brunet wrote:
> Thanks for noticing that Alexandr.  I see this state was added in 1.5 
> and apparently the code from which I borrowed this from was never 
> updated.  I will remove that and open a bug to remove that from the 
> original.  -Pete
>
> On 4/19/16 7:39 AM, Alexander Scherbatiy wrote:
>> On 19/04/16 05:50, Pete Brunet wrote:
>>> Please review the following patch.
>>>
>>> Bug:https://bugs.openjdk.java.net/browse/JDK-8076554
>>> Patch:http://cr.openjdk.java.net/~ptbrunet/JDK-8076554/webrev.02/
>>>
>>> The problem is that the code is currently hardcoded to only support text
>>> accessibility for JTextComponent using the DocumentListener,
>>> CaretListener interfaces.  This eliminates the ability to provide
>>> accessibility for a custom text components.  The resolution is to remove
>>> the JTextComponent gate and use the PropertyChangeListener listener
>>> interface with its ACCESSIBLE_CARET_PROPERTY and
>>> ACCESSIBLE_TEXT_PROPERTY properties.
>>   The AccessibleState class already has the MANAGES_DESCENDANTS 
>> constant. What is purpose to add a constant with the same name to the 
>> _AccessibleState?
>>
>>   Thanks,
>>   Alexandr.
>>> I tested this with the two test cases which can be found at
>>> http://cr.openjdk.java.net/~ptbrunet/JDK-8076554/test/
>>> running them side by side with VoiceOver.
>>>
>>> Note that I will fix the similar JProgressBar and JSlider restrictions
>>> in JDK-8154507 and JDK-8154508 respectively.
>>>
>>> Pete
>>>
>>>
>>
>

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


More information about the swing-dev mailing list