Adjust position of drawers and stickcategoryheader with active smartmenu top bar, resolves #870 #879 #899

[christianwolters]

see #870 , #879 and #899.

[abias]

Hi @christianwolters ,

many thanks for this contribution.

I just rebased the PR onto latest master and made some minor review changes in a separate commit.

Regarding #870, I can confirm that the bug is solved by this patch.

Regarding #879, I think there is some room for improvement.
On the one hand, you have to note that on small screens, the menu bar is placed behind the course index drawer and not above it anymore. But this means that the calculated height of the drawer does not fit anymore on small screens:

Additionally, I would like to point out that we have already some places where the .boost-union-menubar class is used as a selector, especially for handling the height of the message drawer together with the menu bar.

Would you mind to interweave your new handling for the course index with the existing handling for the message drawer and use the same breakpoints? And it would be a great benefit if you could also modify the handling of the message drawer to use the new $boost-union-smartmenu-menubar-height variable as well.

Many thanks,
Alex

[abias]

.

[christianwolters]

Hey @abias,

thanks for pointing this out.

I''ve fixed this for small and middle-size screens (not pushed yet).

During testing, I encountered additonal bugs, when top menubar and footerbar are both enabled:

• I'll create a seperate issue for this
• but will adress this in this PR as well