#32969 closed defect (bug) (fixed)
the Add Media button should be a button
| Reported by: |
|
Owned by: |
|
|---|---|---|---|
| Milestone: | 4.3 | Priority: | normal |
| Severity: | normal | Version: | |
| Component: | Editor | Keywords: | has-patch |
| Focuses: | accessibility, javascript | Cc: |
Description
Currently, the Add Media button is a non-link link (with a href="#" attribute) and has a title attribute that repeats the link text:
For accessibility and semantics should be a button element. The current title attribute is redundant and doesn't add any useful information, should be removed. Worth noting when using ChromeVox it's impossible to open the media modal using the keyboard, whatever you try: pressing Enter or pressing Spacebar does nothing. While this has to do with some specific ChromeVox quirks with URL fragment links, events and preventing the default action, using a button instead of a link would fix it.
Attachments (1)
Change History (6)
#2
@
10 years ago
- Milestone changed from Awaiting Review to Future Release
- Version 4.2 deleted
Looking good.
This ticket was mentioned in Slack in #core-editor by iseulde. View the logs.
10 years ago
Note: See
TracTickets for help on using
tickets.

In the proposed patch: