From de97b87bf7e7126d825a80b4ba86d2ce50f18ca0 Mon Sep 17 00:00:00 2001
From: 1letter <1letter@gmx.de>
Date: Mon, 18 Nov 2024 17:07:42 +0100
Subject: [PATCH 1/3] feat(pat contentbrowser): improvements ui

---
 .../contentbrowser/src/ContentBrowser.svelte  | 20 ++++++++++++++++++-
 1 file changed, 19 insertions(+), 1 deletion(-)

diff --git a/src/pat/contentbrowser/src/ContentBrowser.svelte b/src/pat/contentbrowser/src/ContentBrowser.svelte
index 17550fb3d..3f6565a57 100644
--- a/src/pat/contentbrowser/src/ContentBrowser.svelte
+++ b/src/pat/contentbrowser/src/ContentBrowser.svelte
@@ -342,6 +342,13 @@
         }, 500);
     }
 
+    function initToolTips(){
+        // init the tooltips for select buttons in contentbrowser
+        const tooltipTriggerList = document.querySelectorAll('[data-bs-toggle="tooltip"]')
+        const tooltipList = [...tooltipTriggerList].map(tooltipTriggerEl => new bootstrap.Tooltip(tooltipTriggerEl))
+        console.log(tooltipTriggerList, tooltipList)
+    }
+
     $: {
         if ($showContentBrowser) {
             contentItems.get({ path: $currentPath });
@@ -409,6 +416,7 @@
                         <div
                             class="levelColumn{i % 2 == 0 ? ' odd' : ' even'}"
                             in:fly|local={{ duration: 300 }}
+                            use:initToolTips
                         >
                             <div class="levelToolbar">
                                 {#if i == 0 && $config.mode == "browse"}
@@ -426,11 +434,15 @@
                                 {#if level.selectable}
                                     <button
                                         class="btn btn-primary btn-xs"
+                                        data-bs-toggle="tooltip"
+                                        data-bs-title="{level.displayPath}"
+                                        data-bs-placement="top"
+                                        data-bs-custom-class="contentbrowser-tooltip"
                                         disabled={!isSelectable(level)}
                                         on:click|preventDefault={() => addItem(level)}
                                     >
                                         {_t("select ${level_path}", {
-                                            level_path: level.displayPath,
+                                            level_path: level.Title,
                                         })}
                                     </button>
                                 {/if}
@@ -753,4 +765,10 @@
         text-align: center;
         padding: 0.25rem 0;
     }
+
+    /* override the default tooltip z-index, because contentbrowser is on a higher level */
+    :global(div.tooltip.contentbrowser-tooltip){
+        --bs-tooltip-zindex: 2000;
+        z-index: var(--bs-tooltip-zindex);
+    }
 </style>

From b28f7890b909a295ef4b1a8cc6db3eeaa34327d2 Mon Sep 17 00:00:00 2001
From: 1letter <1letter@gmx.de>
Date: Wed, 4 Dec 2024 09:30:14 +0100
Subject: [PATCH 2/3] feat(pat contentbrowser): use title attribute of select
 button to display path info of selected object

---
 .../contentbrowser/src/ContentBrowser.svelte  | 21 ++-----------------
 1 file changed, 2 insertions(+), 19 deletions(-)

diff --git a/src/pat/contentbrowser/src/ContentBrowser.svelte b/src/pat/contentbrowser/src/ContentBrowser.svelte
index 3f6565a57..05649f352 100644
--- a/src/pat/contentbrowser/src/ContentBrowser.svelte
+++ b/src/pat/contentbrowser/src/ContentBrowser.svelte
@@ -342,13 +342,6 @@
         }, 500);
     }
 
-    function initToolTips(){
-        // init the tooltips for select buttons in contentbrowser
-        const tooltipTriggerList = document.querySelectorAll('[data-bs-toggle="tooltip"]')
-        const tooltipList = [...tooltipTriggerList].map(tooltipTriggerEl => new bootstrap.Tooltip(tooltipTriggerEl))
-        console.log(tooltipTriggerList, tooltipList)
-    }
-
     $: {
         if ($showContentBrowser) {
             contentItems.get({ path: $currentPath });
@@ -416,7 +409,6 @@
                         <div
                             class="levelColumn{i % 2 == 0 ? ' odd' : ' even'}"
                             in:fly|local={{ duration: 300 }}
-                            use:initToolTips
                         >
                             <div class="levelToolbar">
                                 {#if i == 0 && $config.mode == "browse"}
@@ -434,13 +426,10 @@
                                 {#if level.selectable}
                                     <button
                                         class="btn btn-primary btn-xs"
-                                        data-bs-toggle="tooltip"
-                                        data-bs-title="{level.displayPath}"
-                                        data-bs-placement="top"
-                                        data-bs-custom-class="contentbrowser-tooltip"
+                                        title="{level.displayPath}"
                                         disabled={!isSelectable(level)}
                                         on:click|preventDefault={() => addItem(level)}
-                                    >
+                                                                            >
                                         {_t("select ${level_path}", {
                                             level_path: level.Title,
                                         })}
@@ -765,10 +754,4 @@
         text-align: center;
         padding: 0.25rem 0;
     }
-
-    /* override the default tooltip z-index, because contentbrowser is on a higher level */
-    :global(div.tooltip.contentbrowser-tooltip){
-        --bs-tooltip-zindex: 2000;
-        z-index: var(--bs-tooltip-zindex);
-    }
 </style>

From c250e2359501387c5058e838c3334bf07d429081 Mon Sep 17 00:00:00 2001
From: Johannes Raggam <thetetet@gmail.com>
Date: Wed, 4 Dec 2024 10:49:18 +0100
Subject: [PATCH 3/3] Update src/pat/contentbrowser/src/ContentBrowser.svelte

---
 src/pat/contentbrowser/src/ContentBrowser.svelte | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/pat/contentbrowser/src/ContentBrowser.svelte b/src/pat/contentbrowser/src/ContentBrowser.svelte
index 05649f352..1c90751ee 100644
--- a/src/pat/contentbrowser/src/ContentBrowser.svelte
+++ b/src/pat/contentbrowser/src/ContentBrowser.svelte
@@ -429,7 +429,7 @@
                                         title="{level.displayPath}"
                                         disabled={!isSelectable(level)}
                                         on:click|preventDefault={() => addItem(level)}
-                                                                            >
+                                    >
                                         {_t("select ${level_path}", {
                                             level_path: level.Title,
                                         })}