From dbba3496afc79a0c5bc0d2dc55c69d5d9d7bdcff Mon Sep 17 00:00:00 2001
From: "alonso.torres" <alonso.torres@kaleidos.net>
Date: Fri, 2 Jan 2026 11:01:01 +0100
Subject: [PATCH] :bug: Fix problems with trash bar in dashboard

---
 frontend/src/app/main/ui/dashboard.cljs       |  8 +++++-
 .../src/app/main/ui/dashboard/deleted.cljs    | 25 ++++++++++---------
 .../src/app/main/ui/dashboard/deleted.scss    | 24 ++++++++++--------
 .../src/app/main/ui/dashboard/projects.cljs   |  5 +++-
 4 files changed, 37 insertions(+), 25 deletions(-)

diff --git a/frontend/src/app/main/ui/dashboard.cljs b/frontend/src/app/main/ui/dashboard.cljs
index f8ed5aee43..04c376def4 100644
--- a/frontend/src/app/main/ui/dashboard.cljs
+++ b/frontend/src/app/main/ui/dashboard.cljs
@@ -75,7 +75,13 @@
 
         show-templates?
         (and (contains? cf/flags :dashboard-templates-section)
-             (:can-edit permissions))]
+             (:can-edit permissions))
+
+        show-deleted? (:can-edit permissions)
+
+        section (if (and (not show-deleted?) (= section :dashboard-deleted))
+                  :dashboard-recent
+                  section)]
 
     (mf/with-effect []
       (let [key1 (events/listen js/window "resize" on-resize)]
diff --git a/frontend/src/app/main/ui/dashboard/deleted.cljs b/frontend/src/app/main/ui/dashboard/deleted.cljs
index 254ee249d5..6116d8230b 100644
--- a/frontend/src/app/main/ui/dashboard/deleted.cljs
+++ b/frontend/src/app/main/ui/dashboard/deleted.cljs
@@ -190,18 +190,19 @@
            (st/emit! (dcm/go-to-dashboard-deleted :team-id team-id))))]
 
     [:div {:class (stl/css :nav)}
-     [:div {:class [(stl/css :nav-option)
-                    (stl/css-case :selected (= section :dashboard-recent))]
-            :data-testid "recent-tab"
-            :on-click on-recent-click}
-      (tr "labels.recent")]
-     [:div {:class [(stl/css :nav-option)
-                    (stl/css-case :selected (= section :dashboard-deleted))]
-            :variant "ghost"
-            :type "button"
-            :data-testid "deleted-tab"
-            :on-click on-deleted-click}
-      (tr "labels.deleted")]]))
+     [:div {:class (stl/css :nav-inside)}
+      [:div {:class [(stl/css :nav-option)
+                     (stl/css-case :selected (= section :dashboard-recent))]
+             :data-testid "recent-tab"
+             :on-click on-recent-click}
+       (tr "labels.recent")]
+      [:div {:class [(stl/css :nav-option)
+                     (stl/css-case :selected (= section :dashboard-deleted))]
+             :variant "ghost"
+             :type "button"
+             :data-testid "deleted-tab"
+             :on-click on-deleted-click}
+       (tr "labels.deleted")]]]))
 
 (mf/defc deleted-section*
   [{:keys [team projects]}]
diff --git a/frontend/src/app/main/ui/dashboard/deleted.scss b/frontend/src/app/main/ui/dashboard/deleted.scss
index 00cdb3258d..989e350470 100644
--- a/frontend/src/app/main/ui/dashboard/deleted.scss
+++ b/frontend/src/app/main/ui/dashboard/deleted.scss
@@ -4,6 +4,7 @@
 //
 // Copyright (c) KALEIDOS INC
 
+@use "refactor/common-refactor.scss" as deprecated;
 @use "common/refactor/common-dashboard";
 @use "../ds/typography.scss" as t;
 @use "../ds/_borders.scss" as *;
@@ -46,20 +47,21 @@
 }
 
 .nav {
+  background: var(--color-background-default);
+  padding: var(--sp-xxl) var(--sp-xxl) var(--sp-s) var(--sp-xxl);
+  position: sticky;
+  top: 0;
+
+  // We need to use the the deprecated z-index so it won't clash with the dashboard
+  // onboarding modals
+  z-index: deprecated.$z-index-3;
+}
+
+.nav-inside {
+  border-bottom: $b-1 solid var(--panel-border-color);
   display: flex;
   gap: var(--sp-l);
   justify-content: space-between;
-  border-bottom: $b-1 solid var(--panel-border-color);
-  //padding-inline-start: var(--sp-l);
-  background: var(--color-background-default);
-  position: sticky;
-  top: 0;
-  z-index: var(--z-index-panels);
-
-  /* margin: 0 1.5rem; */
-  /* margin-top: 1rem; */
-
-  margin: var(--sp-xxl) var(--sp-xxl) var(--sp-xxl) var(--sp-xxl);
 }
 
 .nav-option {
diff --git a/frontend/src/app/main/ui/dashboard/projects.cljs b/frontend/src/app/main/ui/dashboard/projects.cljs
index af92638e5d..04ca802352 100644
--- a/frontend/src/app/main/ui/dashboard/projects.cljs
+++ b/frontend/src/app/main/ui/dashboard/projects.cljs
@@ -331,6 +331,8 @@
         my-penpot?      (= (:default-team-id profile) team-id)
         default-team?   (:is-default team)
 
+        show-deleted?   (:can-edit permisions)
+
         projects
         (mf/with-memo [projects]
           (->> projects
@@ -378,7 +380,8 @@
                                                           show-team-hero?
                                                           can-invite))}
 
-          [:> deleted/menu* {:team-id team-id :section :dashboard-recent}]
+          (when show-deleted?
+            [:> deleted/menu* {:team-id team-id :section :dashboard-recent}])
 
           (for [{:keys [id] :as project} projects]
             ;; FIXME: refactor this, looks inneficient