#36562 closed defect (bug) (fixed)
Customizer: site icon preview accessibility improvements
Reported by: | afercia | Owned by: | afercia |
---|---|---|---|
Milestone: | 4.6 | Priority: | normal |
Severity: | normal | Version: | 4.4 |
Component: | Customize | Keywords: | has-screenshots |
Focuses: | accessibility | Cc: |
Description
Trying to address some issues noticed when discussing #36450.
The "browser interface" image used for the sidebar control has an alt attribute 'Browser interface preview' that should be removed for consistency with the same image used in the Media dialog.
Also, some text is redundant when using a screen reader:
- the bold text and the alt attributes say the same thing
- the site name is just for visual purposes, should be hidden for assistive technologies
About the image crop preview, the whole cropping task is not accessible at the moment so the best thing to do is probably just use an alt attribute to say what the image is and that cropping requires mouse interaction.
Props from #36450 go to shahpranaf and TacoVerdo.
In the screenshots below, the highlighted text should be hidden for assisitive technologies because it just repeats the images alt attributes or because it is not useful (e.g. the site name).
Attachments (1)
Change History (6)
#2
@
9 years ago
- Owner set to afercia
- Resolution set to fixed
- Status changed from new to closed
In 37229:
#3
@
9 years ago
- Keywords needs-patch added; has-patch removed
- Resolution fixed deleted
- Status changed from closed to reopened
I think the preview in the customizer needs to have the alt attribute, or otherwise hidden text that explains that there's the browser preview and app icon preview. It had alt text because it doesn't have labels like in the modal.
In the proposed patch:
aria-hidden="true"
Image crop area preview. Requires mouse interaction.