Change the default close button & icon elements to use <button> elements, either instead of a div, or as an added wrapper around the <i> icon element.
This is mainly to make these elements focusable, tabbable and clickable with a keyboard. Among other things, this change will improve accessibility for visually-impaired screenreader users, as well as users with reduced motor skills who might prefer to use a keyboard instead of a mouse.
Add 'rel' attribute param to the link option. Allowing developers to add the rel attribute will enable them to set rel="noopener noreferrer" on external links, especially those which open in a new tab. This helps mitigate a securityvulnerability and potentially improves performance too.
requirements
copy pasted from this PR:
<button>elements, either instead of a div, or as an added wrapper around the<i>icon element. This is mainly to make these elements focusable, tabbable and clickable with a keyboard. Among other things, this change will improve accessibility for visually-impaired screenreader users, as well as users with reduced motor skills who might prefer to use a keyboard instead of a mouse.specs
{optional: describe technical specs to implement this feature, if not obvious}
misc
{optional: other useful info}