This Bugzilla instance is a read-only archive of historic NetBeans bug reports. To report a bug in NetBeans please follow the project's instructions for reporting issues.

Bug 128027 - A11Y - Satisfaction Survey dialog
Summary: A11Y - Satisfaction Survey dialog
Status: RESOLVED FIXED
Alias: None
Product: ide
Classification: Unclassified
Component: Welcome (show other bugs)
Version: 6.x
Hardware: All All
: P3 blocker (vote)
Assignee: Denis Anisimov
URL:
Keywords: A11Y
Depends on:
Blocks:
 
Reported: 2008-02-21 14:59 UTC by Lukas Hasik
Modified: 2008-02-29 12:27 UTC (History)
0 users

See Also:
Issue Type: DEFECT
Exception Reporter:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Lukas Hasik 2008-02-21 14:59:01 UTC
-there are missing mnemonics and description

===============================================
 Tested Window title : NetBeans Users Satisfaction Survey
===============================================

Results of Accessibility test


 Doesn't implement Accessible :
   - none.

 No Accessible name :
   - none.

 No Accessible description :
   Class: javax.swing.JButton {  Go to Survey |  }
   Class: javax.swing.JButton {  Remind Later |  }
   Class: javax.swing.JButton {  Skip Survey |  }


 Label with LABEL_FOR not set :
   - none.

 Components with no LABEL_FOR pointing to it :
   - none.

 Components with no mnemonic :
   Class: javax.swing.JButton {  Go to Survey |  }
   Class: javax.swing.JButton {  Remind Later |  }
   Class: javax.swing.JButton {  Skip Survey |  }


 Components with wrong mnemonic (mnemonic isn't ASCII , label doesn't contain mnemonic):
   - none.

 Components not reachable with tab traversal :
   - none.
Comment 1 Denis Anisimov 2008-02-29 08:29:52 UTC
I have fixed issue with mnemonics.
But there is still no accessible description.
But fix for this relates to Dialogs API . And it should be 
other issue. The problem that usage of dialog API
doesn't allow to manage A11Y for buttons that it provides.
So this API should somehow set up accessible description inside 
its implementation . This issue concerns usage of 
setOption(Object[]) method in NotifyDescriptor class.
This method should lead to setting up accessible description
for created buttons in dialog. Either one need provide 
an API method for this.

Possibly this is already fixed and should be rechecked.

changeset:   70204:7f76cb083814
Comment 2 Denis Anisimov 2008-02-29 08:33:21 UTC
forgot to reassign
Comment 3 Denis Anisimov 2008-02-29 08:34:40 UTC
Now closing as fixed.
Comment 4 Stanislav Aubrecht 2008-02-29 08:49:58 UTC
i think it's possible to use accessible enabled JButton instances in setOptions() directly...
Comment 5 Denis Anisimov 2008-02-29 11:35:08 UTC
1) You are wrong.
JButton cannot be set in array with setOptions() method. It's checked.
Please read documentation for this method. JButton will be converted into String via toString() 
and just labels appears in Dialog, not JButtons.
2) Even if such suggestion would be possible there is still bug in Dialogs API becuase in this case
usage of setOptions(String[] ) method will always lead to bad result : there will be no a11y support in resulting buttons.
So why one need such method at all ? Each time when you need good a11y support you will need to use your
own Jcomponent in setOptions() method. It is problem implementation of API or API itself.
3) You suggestion can be applied to DialogDescriptor class. But in this case you will need to provide your own 
content panel and this will lead to change UI. And moreover you are not able to use NotifyDescriptor nested classes
in this case ( when you need to set up your own text for JButtons ).

So each time problems in Dialogs API appears and the best way to fix a lot of related possible problems 
is fix in Dialogs API , not in each usage of this API.
Comment 6 Denis Anisimov 2008-02-29 11:51:29 UTC
Ops, sorry .
I don't understand why in previous time when I checked there was just label instead of button.
You are right , it is possible to set JButton via setOptions.
I will do it .

But I still think that there is issue in impl of Dialogs API because setOptions(String[] ) method
should never be used because of a11y problems.
Comment 7 Denis Anisimov 2008-02-29 12:27:09 UTC
Fully fixed ( access. descr is added also ).
changeset:   70667:5c0828406168