Fix drop menu position with limited vertical screen space

[JammingBen]

Description

This changes the way how drop menus are positioned when only limited vertical screen space is available. They now fully utilize the available vertical screen space.

The issue was noticeable when the available space above and below the drop-trigger-element was not enough to hold the menu. If that's the case, we now move the drop to the top of the current container.

Related Issue

• Fixes [Web] Contexmenu is partly hidden #8411
• Fixes [Web] Firefox: Dropmenu is partly hidden #8412

Types of changes

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to change)
• Technical debt
• Tests

[sonarcloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
1 Code Smell

80.0% Coverage
0.0% Duplication

[kulmann]

I have such an ultra nitpick that I'm nearly ashamed to post it, but here goes nothing.
If you resize your browser window to such a level that the respective OcDrop needs to be made scrollable the hover effect lacks margin on the right 😁 See screenshot.

[JammingBen]

I guess it makes room for a vertical scrollbar - but why is there no vertical scrollbar?! 😄 I can only reproduce that having a vertical scrollbar visible.

[kulmann]

I guess it makes room for a vertical scrollbar - but why is there no vertical scrollbar?! 😄 I can only reproduce that having a vertical scrollbar visible.

I only have the scroll bar visible during scrolling

[JammingBen]

I only have the scroll bar visible during scrolling

W00t. Which browser are you using? I tried FF and Chrome, both fine.

[kulmann]

I only have the scroll bar visible during scrolling

W00t. Which browser are you using? I tried FF and Chrome, both fine.

Chrome

[tbsbdr]

I only have the scroll bar visible during scrolling

W00t. Which browser are you using? I tried FF and Chrome, both fine.

Chrome

cant reproduce
looks good on my side in current FF & Chrome

[kulmann]

I only have the scroll bar visible during scrolling

W00t. Which browser are you using? I tried FF and Chrome, both fine.

Chrome

cant reproduce looks good on my side in current FF & Chrome

Congratulations though, because you did reproduce it. The blue background of the Viewer item is touching the scroll bar.

I didn't request changes with this nitpick, just commented it. No way I'm blocking this pull request for this small detail which is hardly ever relevant on a 720p+ display. ;-) (I also didn't fully review the PR, so that's why I didn't approve it yet)

[kulmann]

😍