From: Daniel Calviño Sánchez <danxuliu@gmail.com>
Date: Thu, 2 Dec 2021 07:01:20 +0000 (+0100)
Subject: Remove no longer needed CSS rules
X-Git-Tag: v23.0.1rc1~39^2
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=refs%2Fpull%2F30144%2Fhead;p=nextcloud-server.git

Remove no longer needed CSS rules

Since Nextcloud 22 the "#new-user" rules had no effect, as the DOM
structure changed to show a dialog rather than adding a row on top of
the list when adding new users.

Similarly, the z-index was no longer needed, as there will be no
"new-user" row that could overlap. Moreover, the z-index was set even
higher (100) in another rule still active.

Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
---

diff --git a/apps/settings/css/settings.scss b/apps/settings/css/settings.scss
index 6b79040ce9e..e186c285e64 100644
--- a/apps/settings/css/settings.scss
+++ b/apps/settings/css/settings.scss
@@ -1555,8 +1555,7 @@ doesnotexist:-o-prefocus, .strengthify-wrapper {
 			}
 
 			/* various */
-			&#grid-header,
-			&#new-user {
+			&#grid-header {
 				@include position('sticky');
 				align-self: normal;
 				background-color: var(--color-main-background);
@@ -1568,18 +1567,8 @@ doesnotexist:-o-prefocus, .strengthify-wrapper {
 				}
 			}
 
-			// separate prop to set initial value to top: 50px
-			&#new-user {
-				height: 120px;
-
-				.row {
-					padding-top: 50px;
-				}
-			}
-
 			&#grid-header {
 				color: var(--color-text-maxcontrast);
-				z-index: 60; /* above new-user */
 				border-bottom-width: thin;
 
 				#headerDisplayName,