[wp-trac] [WordPress Trac] #42979: Draw "dismiss" icon in upload errors in the error div, not after the action title
WordPress Trac
noreply at wordpress.org
Thu Nov 11 19:20:29 UTC 2021
#42979: Draw "dismiss" icon in upload errors in the error div, not after the action
title
--------------------------+------------------------------------------------
Reporter: ComputerGuru | Owner: joedolson
Type: defect (bug) | Status: assigned
Priority: normal | Milestone: 5.9
Component: Media | Version:
Severity: normal | Resolution:
Keywords: has-patch | Focuses: ui, accessibility, administration
needs-testing |
--------------------------+------------------------------------------------
Changes (by joedolson):
* keywords: needs-patch => has-patch needs-testing
Comment:
Placing the dismiss button inside the error div only makes sense if we're
also going to implement separate dismiss buttons for each individual
error. I agree that it's not really beneficial to dismiss each error
individually, so this is a simplest case.
- Changes to a text button, using the existing screen-reader-text.
- Moves the button to the bottom of the status container.
- Updates CSS to remove dashicon & position button.
To test:
- upload one or more disallowed file types in the media library grid & in
the media library modal
- Check positioning and text of dismiss button
- Verify screen reader behavior. (Read errors, then button text, focus
button)
--
Ticket URL: <https://core.trac.wordpress.org/ticket/42979#comment:23>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list