#13740 closed defect (fixed)
dijit.ColorPalette: When Jaws reads the table there is no summary
| Reported by: | Katie Vance | Owned by: | Katie Vance |
|---|---|---|---|
| Priority: | high | Milestone: | 1.7 |
| Component: | Accessibility | Version: | 1.6.1 |
| Keywords: | a11y, jaws | Cc: | Becky Gibson |
| Blocked By: | Blocking: |
Description
When JAWS reads the color palette, it just reads the colors without any explanation.
Attachments (1)
Change History (11)
comment:1 Changed 10 years ago by
comment:2 Changed 10 years ago by
The default is actually no summary. Then if the user needs a summary, they can specify. I didn't want to require a summary because that would break existing widgets.
comment:3 Changed 10 years ago by
Couple things on the patch:
- Can't you just use
_setPaletteSummaryAttr: "paletteTableNode"
rather than that special setting code?
- Like all widget parameters, paletteSummary needs an API doc comment with the description of what it is
Other than that seems good, although I note that things like Editor's foreground/background color pickers still won't announce a summary. Maybe that's OK since they are spawned from a DropDownButton that has a label?
comment:4 Changed 10 years ago by
Fixed patch accordingly. I also tried the color palette in the editor and because the button had a description it was not confusing to the user what the content was in the color palette. I think it is ok that the summary isn't read in those instances.
comment:5 Changed 10 years ago by
| Milestone: | tbd → 1.7 |
|---|
OK thanks, looks good except you need an API comment for summary so it shows up in the API documentation (same as all widget attributes).
About the roles, since this table is composed of actual <table> and <td> nodes, do we need role=grid and role=gridcell at all?
comment:6 Changed 10 years ago by
PS: thanks to #12377 admittedly we don't document "obvious" attributes to widgets like HTML5 or aria settings, not sure if summary falls into that category or not.
comment:7 Changed 10 years ago by
Added summary attribute to the widget. Also added test case for set. I added the roles of grid and gridcell because without them, the colors were not being announced for certain browsers.
Looks good to me as long as JAWS is happy. Although, I wouldn't add in a default summary - make the user responsible for adding the appropriate, localized text. Just my two cents :-)