#35142 closed defect (bug) (fixed)
Remove title attributes and other accessibility improvements for the Authentication Check dialog "Close X"
Reported by: | afercia | Owned by: | afercia |
---|---|---|---|
Milestone: | 4.5 | Priority: | normal |
Severity: | normal | Version: | 4.4 |
Component: | Administration | Keywords: | has-screenshots, has-patch, title-attribute |
Focuses: | ui, accessibility | Cc: |
Description
See related #24766 and all the following tickets about title attributes.
The "Close X" in the authentication check modal dialog has a title attribute that says "Close". It should be replaced with some hidden text, as already done in similar cases in the admin.
Additionally, it's just a <div>
element and should be a button to natively support keyboard interaction and for better semantics.
See screenshot:
Attachments (1)
Change History (5)
Note: See
TracTickets for help on using
tickets.
The proposed patch removes the title attribute and uses a
<button>
instead of a<div>
. Also simplifies the CSS and takes care of the hover/focus style.