Opened 9 years ago
Closed 9 years ago
#34273 closed defect (bug) (fixed)
#a11y-headings - the Media Views headings
Reported by: | afercia | Owned by: | afercia |
---|---|---|---|
Milestone: | 4.4 | Priority: | normal |
Severity: | normal | Version: | 4.3 |
Component: | Media | Keywords: | has-patch |
Focuses: | ui, accessibility | Cc: |
Description
Basically all the headings in the media grid and the media modal should be adjusted to the new headings hierarchy. For example, inspecting the Media Library > grid mode screen with the Firefox Web Developer toolbar, here's the headings printed out in the markup:
The empty H1 and the "Uploading" H3 are actually hidden with display: none
and not used in this screen (they're used in the Media modal). Ideally, they shouldn't be printed out at all if not used, but probably we can live with having them hidden.
By the way probably all the H3 should be bumped up to a H2, both in the grid and in the media modal. For example, here's the "Uploading" heading displayed in the media modal sidebar:
I don't see any reason why it should be a H3 thus skipping a heading level.
Notes:
- the "Drop files anywhere to upload" in the first screenshot gets displayed after the "Add New" button is pressed.
- the "Drop files to upload" is displayed in the "drop zone" overlay when dragging media inside the browser's window, not sure it should be a heading in the first place.
#a11y-headings
Attachments (3)
Change History (16)
#2
@
9 years ago
- Owner set to wonderboymusic
- Status changed from new to assigned
Not sure if there's the need for some CSS backwards compatibility, thinking at plugins that may extend the Media Views and still use H3 headings. Assigning to the Media Views Master for his feedback and review :)
This ticket was mentioned in Slack in #accessibility by afercia. View the logs.
9 years ago
#5
@
9 years ago
- Keywords commit added
Refreshed patch for some CSS back compatibility. cc @wonderboymusic
#6
@
9 years ago
- Owner changed from wonderboymusic to afercia
all yours to commit - keeping it assigned to you in case it gets reopened
#8
@
9 years ago
- Keywords needs-patch added; has-patch commit removed
- Resolution fixed deleted
- Status changed from closed to reopened
The changes to heading levels in [35427] is causing the following message to persistently appear when viewing the Upload Files tab in the media manager:
The web browser on your device cannot be used to upload files. You may be able to use the native app for your device instead.
First pass. Not sure about the
.uploader-window h1
maybe it shouldn't be a heading at all.cc @wonderboymusic :)