Metadata
- Source
- FLUID-3412
- Type
- Bug
- Priority
- Critical
- Status
- Closed
- Resolution
- Fixed
- Assignee
- Laurel Williams
- Reporter
- Laurel Williams
- Created
2009-12-04T14:46:49.000-0500 - Updated
2009-12-10T21:04:42.000-0500 - Versions
- N/A
- Fixed Versions
-
- 1.2
- Component
-
- Infusion Builder
Description
Jacob's redesign looks really great but unfortunately the screen readers do not read the module descriptions that are in the title tags of the labels.
We need to add hidden spans in the labels to handle this.
The current markup is:
<label title="The core of the Fluid Infusion framework. Required for all Fluid components" class="flc-customBuild-inputLabel" for="groups::module::moduleInput">Fluid Infusion Framework</label>
New markup should be something like:
<label title="The core of the Fluid Infusion framework. Required for all Fluid components" class="flc-customBuild-inputLabel" for="groups::module::moduleInput">Fluid Infusion Framework<span class="fl-offScreen-hidden">Fluid Infusion description</span></label>
Comments
-
Laurel Williams commented
2009-12-07T10:02:34.000-0500 because of the accessibility issue I think we should add this task to the Bug Parade Builder 1.1.2
-
Laurel Williams commented
2009-12-08T13:38:17.000-0500 Altered the markup and js to add a hidden span to the checkbox label which contains the module description.
-
Laurel Williams commented
2009-12-09T15:03:17.000-0500 Re-opening to fix two issues:
1) remove flc class on span..not needed
2) programmatically add style from defaults block. -
Laurel Williams commented
2009-12-09T15:36:04.000-0500 removed flc style where it shouldn't have been and added programmatic setting for the hidden style.
-
Colin Clark commented
2009-12-10T21:04:42.000-0500 I've reviewed Laurel's change to the hard-baked class name for the hidden style and it looks good.