moodle-an-hochschulen / moodle-theme_boost_union

Theme Boost Union is an enhanced child theme of Boost which is intended, on the one hand, to make Boost simply more configurable and, on the other hand, to provide helpful additional features for the daily Moodle operation of admins, teachers and students.
GNU General Public License v3.0
55 stars 49 forks source link

Courseindex: icons not part of course module link #588

Open christianwolters opened 3 months ago

christianwolters commented 3 months ago

Via #554 module icons where added to the course index, but they are not part of the link to the course module.

Describe the solution you'd like

Additional context

This is an oversight in the (my) initial design. The icon increases the perceived clickable link area, but serves only informational but no navigational purpose right now.

As part of this the markup for the icon was added here https://github.com/moodle-an-hochschulen/moodle-theme_boost_union/blob/4c0832b06a63f4c3c78471397837a8738bed4644/templates/core_courseformat/local/courseindex/cm.mustache#L55

but the actual link starts a few lines later https://github.com/moodle-an-hochschulen/moodle-theme_boost_union/blob/4c0832b06a63f4c3c78471397837a8738bed4644/templates/core_courseformat/local/courseindex/cm.mustache#L65

it should be tested if a simple reordering of the markup is the way to go.