make add event modal more accessible#745
Conversation
|
This pull request has conflicts ☹ |
|
Hi @rlucke |
|
Use Run test server using develop.opencast.org as backend: Specify a different backend like stable.opencast.org: It may take a few seconds for the interface to spin up. |
|
This pull request is deployed at test.admin-interface.opencast.org/745/2024-06-26_14-13-04/ . |
Arnei
left a comment
There was a problem hiding this comment.
Can access more things by keyboard navigation and the screenreader shouts much less nonsense and more actually useful information, I love it.
Overall very minor issues below:
| {/* Template selection*/} | ||
| <div className="obj tbl-list"> | ||
| <table className="main-tbl"> | ||
| <table className="main-tbl" role="presentation"> |
There was a problem hiding this comment.
Can we use role="none" instead, or is that not yet supported well enough?
|
This pull request has conflicts ☹ |
adding aria-labels, use buttons instead of divs and add some texts for screen readers