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

Internal note: add max height to smart menus #629

Open stefanscholz opened 2 months ago

stefanscholz commented 2 months ago

This is a follow up issue for smart menus and will be considered and spec'd out after the current smart menu enhancements are completed.

Idea: Introduce a max-height setting for each smart menu, with options like:

acquaalta commented 2 weeks ago

Hi @stefanscholz, Thanks for this wonderful theme :-) I noticed something that may be related to the issue that you opened here:

I created a smart menu that is supposed to show the courses that a user is enrolled too, and then I viewed it while being logged in as a user who is enrolled to approximately 30 courses. It seems that in such a scenario, two things happen, at least on some screens: 1) Some courses in the menu are displayed outside its borders, below the smart menu. 2) Because the list of courses in the menu is too long, and there is no scroll bar, it is hard to navigate in a small screen to the courses at the bottom of the menu.

Is it a known issue that the suggested max height to smart menu is supposed to fix? And how, actually? What would happen if the max height would be set to a specific value, and the list of items in the smart menu would exceed that height?

Thanks :-)