fix(NcHeaderMenu): Remove padding from popover menu #6026
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
☑️ Resolves
There should be no padding within the popover menu. The popover container is not style-able (at least not without hacking the (not exposed-)CSS classes), so users can not remove the padding themself.
The padding is a problem if the content is scroll-able, because the scrollbar should be on the very end of the container and not floating around.
🖼️ Screen recording
🏚️ Before
Bildschirmaufnahme_20240830_002829.webm
🏡 After
Bildschirmaufnahme_20240830_002718.webm
🏁 Checklist
next
requested with a Vue 3 upgrade