Main-nav: Fix close button divider (separator) position

With the improvements from db10c45 #10027 and its follow ups, it is now needed to update close button divider margin so, it goes back on being centered aligned Also update code comments around that. Signed-off-by: Pedro Pinto Silva <[email protected]> Change-Id: Ie3fac7e7131c46eb165b2c2d07eef97bfdbce4ce
CollaboraOnline · Nov 29, 2024 · 6af2cec · 6af2cec
1 parent 653107b
commit 6af2cec
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/browser/css/toolbar.css b/browser/css/toolbar.css
@@ -330,16 +330,16 @@
 	width: var(--btn-size);
 	height: var(--btn-size);
 	background-color: transparent;
-	/* same margin as sidebar buttons */
+	/* same margin as main-nav padding-inline-start */
 	margin-inline-end: 5px;
 }
 
 .main-nav.hasnotebookbar #closebuttonwrapperseparator {
 	background: var(--color-background-darker);
 	width: 1px;
 	height: 14px;
-	/* same margin as sidebar buttons */
-	margin-inline-end: 5px;
+	/* same margin as sidebar buttons' padding */
+	margin-inline: 4px;
 }
 
 .main-nav:not(.hasnotebookbar) ~ #closebuttonwrapper {