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
61 stars 56 forks source link

Improvement: Do not add the advertisement tiles div to the frontpage if no tile is activated #516

Closed abias closed 9 months ago

abias commented 9 months ago

During the review of #391 , it became apparent to me that up to now, this (empty) advertisement tile is added to the frontpage even if no tile is activated in Boost Union's setting:

<div id="themeboostunionadvtiles" class="container-fluid"><div class="row"></div></div>

Adding a dediated kill switch to the advertisement tiles does not make much sense as each tile can be deactivated already individually. It just would be good to add the #themeboostunionadvtiles div only if at least one tile is shown.

abias commented 9 months ago

This was solved by #517