<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