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 89455 - A11Y: Background Color dialog has accessibility issues
Summary: A11Y: Background Color dialog has accessibility issues
Status: VERIFIED FIXED
Alias: None
Product: uml
Classification: Unclassified
Component: General (show other bugs)
Version: 5.x
Hardware: All All
: P2 blocker (vote)
Assignee: Yang Su
URL:
Keywords: A11Y
Depends on:
Blocks:
 
Reported: 2006-11-16 13:16 UTC by Sergey Petrov
Modified: 2007-06-08 12:57 UTC (History)
1 user (show)

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 Sergey Petrov 2006-11-16 13:16:15 UTC
reproducible with 061113_2
dialog can be invoked from Class context menu (on diagram): Class|Background Color
Comment 1 Sergey Petrov 2006-11-16 13:24:03 UTC
No Accessible name :
   Class: javax.swing.JTabbedPane {  |  } 


 No Accessible description :
   Class: javax.swing.ColorChooserDialog {   |  } 
   Class: javax.swing.JButton {  Cancel |  } 
   Class: javax.swing.JButton {  OK |  } 
   Class: javax.swing.JButton {  Reset |  } 
   Class: javax.swing.JTabbedPane {  |  } 


Comment 2 Yang Su 2006-11-17 00:09:29 UTC
What we can do here is really just adding a dialog title that tells it's a color
chooser dialog, those buttons don't need any description, the name says it all.
The JTabbedPane is out of our control.

I don't think it's necessary to go with the approach to take JColorChooser panel
and display with NB dialog, it only gives us the control over buttons and dialog
description, and as I said, those components have unambiguous accessible names,
they are fine. 

Dialog title has been added.
Comment 3 Sergey Petrov 2006-11-17 09:38:43 UTC
yes, title is good, but according to netbeans absent of a11y description is an
important issue.
Comment 4 Yang Su 2006-11-17 19:55:48 UTC
OK, we may have gone too far on this whole a11y exercise.

Let's get to the bottom of the problem. Why do we care about accessible name and
description? To make our tool accessible to the visually impaired by providing
descriptions of the components that will receive focus. A component receives
focus when a user or AT reader selects it, usually via keyboard controls.

AccessibleContext() defines two methods that can be used to provide accessible
information:

    * setAccessibleName(String name) sets the name to associate with a given
Accessible object. Typically, an assistive reader will present this name
whenever a component receives focus.

    * setAccessibleDescription(String description) sets the description to
associate with a given Accessible object. Typically, an assistive reader will
present this description when the user asks for more details about a component.

Having a description is always better when the name is considered insufficient
to describe the component, which I'm not convinced is the case here. When we
look at OK, Cancel, Reset buttons do we have questions on what those buttons are
for? 

I hope we have a common understanding on this issue now, and move on. I suggest
we close all those color chooser dialog bugs, leave one ( or create one ) to
depend on the JDK fix for the tracking purpose.
Comment 5 Peter Lam 2007-05-21 20:08:36 UTC
dialog title has been added so this is no longer a p1 issue.
Comment 6 Peter Lam 2007-05-21 20:15:21 UTC
If the dialog or a component name is obvious, a description is not necessary.
For this issue, the title for the Background Color and Border Color dialogs have
the same name "Color Chooser", I'd suggest to add a simple description to
indicate that this it's a color chooser for background for one and border for
the other. If this can not be fixed now, it can be marked for future in the
target milestone.
Comment 7 Peter Lam 2007-05-22 16:43:03 UTC
Change dialog name to "Background Color Chooser".
Comment 8 Yang Su 2007-05-22 18:43:12 UTC
ok, changed dialog title to be more specific
Comment 9 Peter Lam 2007-05-23 23:08:40 UTC
verified in build netbeans-6.0-nightly-200705161200.