From f4f95b27cff88153c14177e2eb51596579836f51 Mon Sep 17 00:00:00 2001
From: Pedro Pinto Silva <pedro.silva@collabora.com>
Date: Fri, 2 Jun 2023 15:30:19 +0200
Subject: [PATCH] Navigator: do not resort to absolute position to set deck

- No element should have border
- Reset max-height otherwise it will inherit 150px

This is a follow up on:
navigator: hide panel title in sidebar 56ca9b6

Signed-off-by: Pedro Pinto Silva <pedro.silva@collabora.com>
Change-Id: I908468f0cb9a494acd7785d76e1882c3a8673de3
---
 browser/css/jsdialogs.css |  2 ++
 browser/css/jssidebar.css | 12 ++++--------
 2 files changed, 6 insertions(+), 8 deletions(-)

diff --git a/browser/css/jsdialogs.css b/browser/css/jsdialogs.css
index aba09e546..422a970aa 100644
--- a/browser/css/jsdialogs.css
+++ b/browser/css/jsdialogs.css
@@ -358,6 +358,8 @@ td.jsdialog > [id^='table-box']:not(.sidebar) {
 	color: var(--color-text-lighter) !important;
 }
 
+/*.ui-treeview-body needs this max-height both in dialogs
+ and within sidebar e.g.: animation > effect*/
 .ui-treeview-body,
 #custom_animation_label_parent {
 	color: var(--color-main-text);
diff --git a/browser/css/jssidebar.css b/browser/css/jssidebar.css
index c55077e4b..50266eacc 100644
--- a/browser/css/jssidebar.css
+++ b/browser/css/jssidebar.css
@@ -328,14 +328,7 @@ button#button2.ui-pushbutton.jsdialog.sidebar {
 
 /* Navigator */
 
-#NavigatorDeck #contentbox,
-#NavigatorDeck #contenttree,
-#NavigatorDeck #tree {
-	position: absolute;
-	top: 30px;
-	bottom: 10px;
-	left: 10px;
-	right: 10px;
+#NavigatorDeck * {
 	border: none;
 }
 #NavigatorDeck .ui-treeview-entry {
@@ -350,6 +343,9 @@ button#button2.ui-pushbutton.jsdialog.sidebar {
 	/* good to move this into css var*/
 	font-size: 14px;
 }
+#NavigatorDeck #contenttree > .ui-treeview-body {
+	max-height: initial;
+}
 #NavigatorDeck .ui-treeview-cell {
 	/* Move this whole block away and fix in the main control */
 	display: flex;