Fix_#113: Added class aalink to coursenamelink in coursecards

trema-tech / moodle-theme_trema

Theme trema, a Moodle LMS plugin.

https://moodle.org/plugins/theme_trema

GNU General Public License v3.0

40 stars 20 forks source link

Fix_#113: Added class aalink to coursenamelink in coursecards #114

Closed dermuck closed 2 years ago

dermuck commented 2 years ago

The coursenamelink within a coursecard now always contains HTML-Attribute class="aalink". This causes the browser to properly highlight the link when he gets focussed (as most other links in moodle), which is important for the use of keyboard navigation.

rmady commented 2 years ago

Great contrib @dermuck Thank you very much ^^ I'll add this new feature for master (4.0) ASAP!

dermuck commented 2 years ago

Thanks @rmady for merging! Glad I can help.

I'm new to this stuff, so maybe its better to ask first: I might provide some more small accessibility fixes for moodle 3.11 (sorry, no moodle 4 around here...) in the future. Would you prefer an All-In-One pull request or should I create separate requests for each fix?

michael-milette commented 2 years ago

Hi @rmady ,

This fix did not make it into the version 4.0 of Trema. If it is still required, could you please add it in?

Best regards,

Michael