Fix setting menu height

- Increase max height to match page height - Move settings 8px from the right instead of 5px to match other top menus - Handle overflow of the content correctly Signed-off-by: Carl Schwan <carl@carlschwan.eu>
nextcloud · Sep 9, 2022 · b155adb · b155adb
1 parent 76f42e1
commit b155adb
Show file tree

Hide file tree

Showing 5 changed files with 13 additions and 10 deletions.
diff --git a/core/css/header.css b/core/css/header.css
diff --git a/core/css/header.css.map b/core/css/header.css.map
diff --git a/core/css/header.scss b/core/css/header.scss
@@ -62,7 +62,7 @@
 
 @mixin header-menu-height() {
 	min-height: calc(44px * 1.5); // show at least 1.5 entries
-	max-height: calc(100vh - #{variables.$header-height} * 2);
+	max-height: calc(100vh - #{variables.$header-height} - 8px);
 }
 
 #header {
@@ -78,9 +78,10 @@
 		position: absolute;
 		max-width: 350px;
 		@include header-menu-height();
-		right: 5px; // relative to parent
+		right: 8px; // relative to parent
 		top: variables.$header-height;
 		margin: 0;
+		overflow-y: scroll;
 
 		&:not(.popovermenu) {
 			display: none;

diff --git a/core/css/server.css b/core/css/server.css