From 23e398d327bc497afa7dcbd3435e45e905345b1b Mon Sep 17 00:00:00 2001
From: Henry Wilkinson <henry@wilkinson.graphics>
Date: Mon, 1 May 2023 03:26:34 -0400
Subject: [PATCH 1/3] Icon updates

- Changes `trash` for `trash3` which I believe wasn't originally available in the version of bootstrap-icons we were using but now it is and I like the tapered edges better :P
- Makes browser profiles action button small to fit with the rest of the dropdown components used elswhere
- Changes previous file-earmark delete icon to trash icons used everywhere else for delete actions
---
 frontend/src/pages/org/browser-profiles-detail.ts | 6 +++---
 frontend/src/pages/org/browser-profiles-list.ts   | 2 +-
 frontend/src/pages/org/crawl-detail.ts            | 2 +-
 frontend/src/pages/org/workflow-detail.ts         | 2 +-
 frontend/src/pages/org/workflows-list.ts          | 2 +-
 5 files changed, 7 insertions(+), 7 deletions(-)

diff --git a/frontend/src/pages/org/browser-profiles-detail.ts b/frontend/src/pages/org/browser-profiles-detail.ts
index 1ccfed13..64472474 100644
--- a/frontend/src/pages/org/browser-profiles-detail.ts
+++ b/frontend/src/pages/org/browser-profiles-detail.ts
@@ -269,7 +269,7 @@ export class BrowserProfilesDetail extends LiteElement {
   private renderMenu() {
     return html`
       <sl-dropdown placement="bottom-end" distance="4">
-        <sl-button slot="trigger" caret>${msg("Actions")}</sl-button>
+        <sl-button size="small" slot="trigger" caret>${msg("Actions")}</sl-button>
 
         <ul
           class="text-left text-sm text-neutral-800 bg-white whitespace-nowrap"
@@ -285,7 +285,7 @@ export class BrowserProfilesDetail extends LiteElement {
           >
             <sl-icon
               class="inline-block align-middle px-1"
-              name="pencil-square"
+              name="pencil"
             ></sl-icon>
             <span class="inline-block align-middle pr-2"
               >${msg("Edit name & description")}</span
@@ -314,7 +314,7 @@ export class BrowserProfilesDetail extends LiteElement {
           >
             <sl-icon
               class="inline-block align-middle px-1"
-              name="file-earmark-x"
+              name="trash3"
             ></sl-icon>
             <span class="inline-block align-middle pr-2">${msg("Delete")}</span>
           </li>
diff --git a/frontend/src/pages/org/browser-profiles-list.ts b/frontend/src/pages/org/browser-profiles-list.ts
index 2589efcc..950bd1f6 100644
--- a/frontend/src/pages/org/browser-profiles-list.ts
+++ b/frontend/src/pages/org/browser-profiles-list.ts
@@ -178,7 +178,7 @@ export class BrowserProfilesList extends LiteElement {
           >
             <sl-icon
               class="inline-block align-middle px-1"
-              name="file-earmark-x"
+              name="trash3"
             ></sl-icon>
             <span class="inline-block align-middle pr-2">${msg("Delete")}</span>
           </li>
diff --git a/frontend/src/pages/org/crawl-detail.ts b/frontend/src/pages/org/crawl-detail.ts
index 531f504e..e751d584 100644
--- a/frontend/src/pages/org/crawl-detail.ts
+++ b/frontend/src/pages/org/crawl-detail.ts
@@ -440,7 +440,7 @@ export class CrawlDetail extends LiteElement {
                 style="--sl-color-neutral-700: var(--danger)"
                 @click=${() => this.deleteCrawl()}
               >
-                <sl-icon name="trash" slot="prefix"></sl-icon>
+                <sl-icon name="trash3" slot="prefix"></sl-icon>
                 ${msg("Delete Crawl")}
               </sl-menu-item>
             `
diff --git a/frontend/src/pages/org/workflow-detail.ts b/frontend/src/pages/org/workflow-detail.ts
index 650a855b..3e6dd408 100644
--- a/frontend/src/pages/org/workflow-detail.ts
+++ b/frontend/src/pages/org/workflow-detail.ts
@@ -473,7 +473,7 @@ export class WorkflowDetail extends LiteElement {
             @click=${() =>
               shouldDeactivate ? this.deactivate() : this.delete()}
           >
-            <sl-icon name="trash" slot="prefix"></sl-icon>
+            <sl-icon name="trash3" slot="prefix"></sl-icon>
             ${
               shouldDeactivate
                 ? msg("Deactivate Workflow")
diff --git a/frontend/src/pages/org/workflows-list.ts b/frontend/src/pages/org/workflows-list.ts
index 30873540..f7e6882a 100644
--- a/frontend/src/pages/org/workflows-list.ts
+++ b/frontend/src/pages/org/workflows-list.ts
@@ -443,7 +443,7 @@ export class WorkflowsList extends LiteElement {
                 ? this.deactivate(workflow)
                 : this.delete(workflow)}
           >
-            <sl-icon name="trash" slot="prefix"></sl-icon>
+            <sl-icon name="trash3" slot="prefix"></sl-icon>
             ${shouldDeactivate
               ? msg("Deactivate Workflow")
               : msg("Delete Workflow")}

From 6e921cc0656208b79b7a59d84f233c24171a3cdf Mon Sep 17 00:00:00 2001
From: Henry Wilkinson <henry@wilkinson.graphics>
Date: Mon, 1 May 2023 03:26:57 -0400
Subject: [PATCH 2/3] Add margin to crawls list

Mirrors workflow list
---
 frontend/src/components/crawl-list.ts | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/frontend/src/components/crawl-list.ts b/frontend/src/components/crawl-list.ts
index fcf9c65f..81086723 100644
--- a/frontend/src/components/crawl-list.ts
+++ b/frontend/src/components/crawl-list.ts
@@ -451,6 +451,8 @@ export class CrawlList extends LitElement {
         border: 1px solid var(--sl-panel-border-color);
         border-radius: var(--sl-border-radius-medium);
         overflow: hidden;
+        margin-left: var(--row-offset);
+        margin-right: var(--row-offset);
       }
 
       .row {

From 088d6d306a37673f61ce097071e8931196a5b9a6 Mon Sep 17 00:00:00 2001
From: Henry Wilkinson <henry@wilkinson.graphics>
Date: Mon, 1 May 2023 03:27:59 -0400
Subject: [PATCH 3/3] Adds hoist to browser profile list actions dropdown

- Should fix bug where you can see the icon buttons through the dropdown
---
 frontend/src/pages/org/browser-profiles-list.ts | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/frontend/src/pages/org/browser-profiles-list.ts b/frontend/src/pages/org/browser-profiles-list.ts
index 950bd1f6..ec773683 100644
--- a/frontend/src/pages/org/browser-profiles-list.ts
+++ b/frontend/src/pages/org/browser-profiles-list.ts
@@ -139,7 +139,7 @@ export class BrowserProfilesList extends LiteElement {
 
   private renderMenu(data: Profile) {
     return html`
-      <sl-dropdown @click=${(e: Event) => e.preventDefault()}>
+      <sl-dropdown hoist="true" @click=${(e: Event) => e.preventDefault()}>
         <sl-icon-button
           slot="trigger"
           name="three-dots"