<Swing Dev> Fwd: The file list in JFileChooser does not have an accessible name.

Pavel Porvatov pavel.porvatov at oracle.com
Tue Aug 30 11:17:42 UTC 2011


Hi Charles,
> On 08/25/2011 05:45 PM, Pavel Porvatov wrote:
>> Hi Charles,
>>> On 07/26/2011 08:58 PM, Charles Lee wrote: Hi guys,
>>>
>>> Could some one take some to look at this patch?
>> I took a look at the patch and have several comments:
>>
>> 1. There is no AccessibleDescription for all resources, therefore I'd 
>> prefer to remove filesAccessibleDescription
>>
>> 2.  Please follow our code conventions 
>> http://www.oracle.com/technetwork/java/codeconvtoc-136057.html
>> a.
>> private String filesListAccessibleName=null;
>> =>
>> private String filesListAccessibleName = null;
>> b. Don't use TABs characters at all
>> c.
>> filesListAccessibleName 
>> =UIManager.getString("FileChooser.filesListAccessibleName",l);
>> =>
>> filesListAccessibleName = 
>> UIManager.getString("FileChooser.filesListAccessibleName", l);
>>
>> etc.
>>
>> 3. As in the last CR fixing you should add the same resources to all 
>> basic_XX.properties files
>>
>> 4. Why don't you use 
>> putClientProperty(AccessibleContext.ACCESSIBLE_NAME_PROPERTY, 
>> accessibleName) instead of "javax.accessibility.AccessibleContext 
>> ac=list.getAccessibleContext();......"
>>
>> Regards, Pavel
> Hi Pavel,
>
> I have modified the patch (attached) according to your comments. Sorry 
> I can not use putClientProperty because it does not work :-)
I took a look at the patch and have several comments:
1. There is no AccessibleDescription for all resources, therefore I'd 
prefer to remove filesAccessibleDescription
2.  Please follow our code conventions. Space is needed after commas in 
args:
filesListAccessibleName = 
UIManager.getString("FileChooser.filesListAccessibleName",l);
=>
filesListAccessibleName = 
UIManager.getString("FileChooser.filesListAccessibleName", l);
3. "Sorry I can not use putClientProperty because it does not work :-)". 
I've checked that it works
4. It seems to complete fix we should add Accessible Name for JTable in 
the FilePane#createDetailsView method

BTW: do you have a case where you need accessible name for JList from 
JFileChooser?

Regards, Pavel
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/swing-dev/attachments/20110830/0256aa41/attachment.html>


More information about the swing-dev mailing list