Merge pull request #35517 from nextcloud/backport/35484/stable25

[stable25] fix the popovermenu
nextcloud · Dec 8, 2022 · 978d669 · 978d669
2 parents 9c79197 + d4b4135
commit 978d669
Show file tree

Hide file tree

Showing 8 changed files with 11 additions and 81 deletions.
diff --git a/apps/files_external/css/settings.css b/apps/files_external/css/settings.css
diff --git a/apps/files_external/css/settings.css.map b/apps/files_external/css/settings.css.map
diff --git a/apps/files_external/css/settings.scss b/apps/files_external/css/settings.scss
@@ -16,6 +16,10 @@
 	}
 }
 
+#externalStorage .popovermenu li > .menuitem {
+    width: fit-content !important;
+}
+
 #externalStorage td.status {
 	/* overwrite conflicting core styles */
 	display: table-cell;

diff --git a/core/css/apps.css b/core/css/apps.css
diff --git a/core/css/apps.css.map b/core/css/apps.css.map
diff --git a/core/css/apps.scss b/core/css/apps.scss
@@ -1218,28 +1218,6 @@ $outter-margin: math.div($popoveritem-height - $popovericon-size, 2);
 	}
 }
 
-/* "app-*" descendants use border-box sizing, so the height of the icon must be
- * set to the height of the item (as well as its width to make it squared). */
-#content[class*='app-'] {
-	.bubble,
-	.app-navigation-entry-menu,
-	.popovermenu {
-		li {
-			> button,
-			> a,
-			> .menuitem {
-				width: fit-content;
-				/* DEPRECATED! old img in popover fallback
-				* TODO: to remove */
-				> img {
-					width: $popoveritem-height;
-					height: $popoveritem-height;
-				}
-			}
-		}
-	}
-}
-
 /* CONTENT LIST ------------------------------------------------------------ */
 .app-content-list {
 	position: -webkit-sticky;