From 19663d958706cb997b2b3dbeeee0c8d18870cf0c Mon Sep 17 00:00:00 2001
From: Daniel James Smith <2670567+djsmith85@users.noreply.github.com>
Date: Mon, 2 Dec 2024 20:26:13 +0100
Subject: [PATCH] [PM-15418] Remove bitItem and use plain bitLink button for
 opening generator history (#12208)

* Remove bitItem and use plain bitLink button for opening generator history

* Revert desktop back to use bitItem

---------

Co-authored-by: Daniel James Smith <djsmith85@users.noreply.github.com>
---
 .../credential-generator.component.html       |  1 +
 .../credential-generator.component.ts         | 10 +--------
 .../credential-generator.component.html       | 21 ++++++++-----------
 .../credential-generator.component.ts         |  4 ++--
 4 files changed, 13 insertions(+), 23 deletions(-)

diff --git a/apps/desktop/src/app/tools/generator/credential-generator.component.html b/apps/desktop/src/app/tools/generator/credential-generator.component.html
index 37d677472d9..66b0de13dfb 100644
--- a/apps/desktop/src/app/tools/generator/credential-generator.component.html
+++ b/apps/desktop/src/app/tools/generator/credential-generator.component.html
@@ -6,6 +6,7 @@
       <button
         type="button"
         bitLink
+        linkType="primary"
         bit-item-content
         aria-haspopup="true"
         (click)="openHistoryDialog()"
diff --git a/apps/desktop/src/app/tools/generator/credential-generator.component.ts b/apps/desktop/src/app/tools/generator/credential-generator.component.ts
index 3acdf76368d..aed8bf18684 100644
--- a/apps/desktop/src/app/tools/generator/credential-generator.component.ts
+++ b/apps/desktop/src/app/tools/generator/credential-generator.component.ts
@@ -17,15 +17,7 @@ import {
   standalone: true,
   selector: "credential-generator",
   templateUrl: "credential-generator.component.html",
-  imports: [
-    DialogModule,
-    ButtonModule,
-    JslibModule,
-    GeneratorModule,
-    ItemModule,
-    ButtonModule,
-    LinkModule,
-  ],
+  imports: [DialogModule, ButtonModule, JslibModule, GeneratorModule, ItemModule, LinkModule],
 })
 export class CredentialGeneratorComponent {
   constructor(private dialogService: DialogService) {}
diff --git a/apps/web/src/app/tools/credential-generator/credential-generator.component.html b/apps/web/src/app/tools/credential-generator/credential-generator.component.html
index 5df4a247449..72e521a0a97 100644
--- a/apps/web/src/app/tools/credential-generator/credential-generator.component.html
+++ b/apps/web/src/app/tools/credential-generator/credential-generator.component.html
@@ -2,16 +2,13 @@
 
 <bit-container>
   <tools-credential-generator />
-  <bit-item>
-    <button
-      type="button"
-      bitLink
-      bit-item-content
-      aria-haspopup="true"
-      (click)="openHistoryDialog()"
-    >
-      {{ "generatorHistory" | i18n }}
-      <i slot="end" class="bwi bwi-angle-right" aria-hidden="true"></i>
-    </button>
-  </bit-item>
+  <button
+    bitLink
+    type="button"
+    linkType="primary"
+    aria-haspopup="true"
+    (click)="openHistoryDialog()"
+  >
+    {{ "generatorHistory" | i18n }}
+  </button>
 </bit-container>
diff --git a/apps/web/src/app/tools/credential-generator/credential-generator.component.ts b/apps/web/src/app/tools/credential-generator/credential-generator.component.ts
index f252796d062..8d7b56a09ad 100644
--- a/apps/web/src/app/tools/credential-generator/credential-generator.component.ts
+++ b/apps/web/src/app/tools/credential-generator/credential-generator.component.ts
@@ -1,6 +1,6 @@
 import { Component } from "@angular/core";
 
-import { ButtonModule, DialogService, ItemModule, LinkModule } from "@bitwarden/components";
+import { ButtonModule, DialogService, LinkModule } from "@bitwarden/components";
 import {
   CredentialGeneratorHistoryDialogComponent,
   GeneratorModule,
@@ -13,7 +13,7 @@ import { SharedModule } from "../../shared";
   standalone: true,
   selector: "credential-generator",
   templateUrl: "credential-generator.component.html",
-  imports: [SharedModule, HeaderModule, GeneratorModule, ItemModule, ButtonModule, LinkModule],
+  imports: [SharedModule, HeaderModule, GeneratorModule, ButtonModule, LinkModule],
 })
 export class CredentialGeneratorComponent {
   constructor(private dialogService: DialogService) {}