User tests: Successful: Unsuccessful:
Pull Request for Issue #20499 .
Fix footer button markup in modal to be consistent with other modals.
Go to Users > User Notes
Create a new note
Click on the User icon (See Cancel button)
Click on the Select button (See Close button)
or
Code review via page source
Before:
<a type="button" class="btn btn-secondary" data-dismiss="modal">Cancel</a>
After:
<a role="button" class="btn btn-secondary" data-dismiss="modal" aria-hidden="true">Close</a>
Buttons markup to be the same.
Buttons markup are different.
none
Status | New | ⇒ | Pending |
Category | ⇒ | Layout |
<button type="button" ...
type="button"
is the critical part here as it will not interfere with form submission (default behaviour on FF and Chrome)
Labels |
Added:
?
|
Status | Pending | ⇒ | Closed |
Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2018-07-16 13:30:11 |
Closed_By | ⇒ | Quy |
Closing in favor for PR #20682
If it is to be a button, use the
button
tag, not the anchor.An anchor with a role="button" is as bad as an anchor with a type="button" attribute.