From 093095e79c2184e05c7e2687ddd7ade1fa1540ee Mon Sep 17 00:00:00 2001
From: Ferdinand Thiessen <opensource@fthiessen.de>
Date: Fri, 30 Aug 2024 00:23:30 +0200
Subject: [PATCH] fix(NcHeaderMenu): Remove padding from popover menu

There should be no padding within the popover menu.
The popover container is not styleable (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 scrollable,
because the scrollbar should be on the very end of the container and not floating around.

Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
---
 src/components/NcHeaderMenu/NcHeaderMenu.vue | 11 +++++------
 1 file changed, 5 insertions(+), 6 deletions(-)

diff --git a/src/components/NcHeaderMenu/NcHeaderMenu.vue b/src/components/NcHeaderMenu/NcHeaderMenu.vue
index f012387670..b4c17b0e1e 100644
--- a/src/components/NcHeaderMenu/NcHeaderMenu.vue
+++ b/src/components/NcHeaderMenu/NcHeaderMenu.vue
@@ -341,6 +341,11 @@ $externalMargin: 8px;
 		// header is filled with primary or image background
 		filter: none !important;
 		color: var(--color-background-plain-text, var(--color-primary-text)) !important;
+
+		&:focus-visible {
+			outline: none !important;
+			box-shadow: none !important;
+		}
 	}
 
 	&--opened &__trigger,
@@ -350,11 +355,6 @@ $externalMargin: 8px;
 		opacity: 1;
 	}
 
-	#{&}__trigger:focus-visible {
-		outline: none !important;
-		box-shadow: none !important;
-	}
-
 	&__wrapper {
 		position: fixed;
 		z-index: 2000;
@@ -362,7 +362,6 @@ $externalMargin: 8px;
 		inset-inline-end: 0;
 		box-sizing: border-box;
 		margin: 0 $externalMargin;
-		padding: 8px;
 		border-radius: 0 0 var(--border-radius) var(--border-radius);
 		border-radius: var(--border-radius-large);
 		background-color: var(--color-main-background);