diff --git a/apps/web/src/app/vault/components/vault-items/vault-cipher-row.component.ts b/apps/web/src/app/vault/components/vault-items/vault-cipher-row.component.ts index 9872ef0da0..ea75743a8c 100644 --- a/apps/web/src/app/vault/components/vault-items/vault-cipher-row.component.ts +++ b/apps/web/src/app/vault/components/vault-items/vault-cipher-row.component.ts @@ -1,5 +1,4 @@ -import { Component, EventEmitter, HostBinding, HostListener, Input, Output } from "@angular/core"; -import { ActivatedRoute, Router } from "@angular/router"; +import { Component, EventEmitter, Input, Output } from "@angular/core"; import { Organization } from "@bitwarden/common/admin-console/models/domain/organization"; import { CipherType } from "@bitwarden/common/vault/enums"; @@ -35,16 +34,6 @@ export class VaultCipherRowComponent { protected CipherType = CipherType; - constructor( - private router: Router, - private activatedRoute: ActivatedRoute, - ) {} - - @HostBinding("class") - get classes() { - return [].concat(this.disabled ? [] : ["tw-cursor-pointer"]); - } - protected get showTotpCopyButton() { return ( (this.cipher.login?.hasTotp ?? false) && @@ -56,16 +45,6 @@ export class VaultCipherRowComponent { return this.cipher.hasOldAttachments && this.cipher.organizationId == null; } - @HostListener("click") - protected click() { - // FIXME: Verify that this floating promise is intentional. If it is, add an explanatory comment and ensure there is proper error handling. - // eslint-disable-next-line @typescript-eslint/no-floating-promises - this.router.navigate([], { - queryParams: { itemId: this.cipher.id }, - queryParamsHandling: "merge", - }); - } - protected copy(field: "username" | "password" | "totp") { this.onEvent.emit({ type: "copyField", item: this.cipher, field }); } diff --git a/apps/web/src/app/vault/components/vault-items/vault-collection-row.component.ts b/apps/web/src/app/vault/components/vault-items/vault-collection-row.component.ts index 73148c64d2..666bec7a1a 100644 --- a/apps/web/src/app/vault/components/vault-items/vault-collection-row.component.ts +++ b/apps/web/src/app/vault/components/vault-items/vault-collection-row.component.ts @@ -1,5 +1,4 @@ -import { Component, EventEmitter, HostBinding, HostListener, Input, Output } from "@angular/core"; -import { Router } from "@angular/router"; +import { Component, EventEmitter, Input, Output } from "@angular/core"; import { Organization } from "@bitwarden/common/admin-console/models/domain/organization"; import { I18nService } from "@bitwarden/common/platform/abstractions/i18n.service"; @@ -38,15 +37,7 @@ export class VaultCollectionRowComponent { @Input() checked: boolean; @Output() checkedToggled = new EventEmitter(); - constructor( - private router: Router, - private i18nService: I18nService, - ) {} - - @HostBinding("class") - get classes() { - return [].concat(this.disabled ? [] : ["tw-cursor-pointer"]); - } + constructor(private i18nService: I18nService) {} get collectionGroups() { if (!(this.collection instanceof CollectionAdminView)) { @@ -71,16 +62,6 @@ export class VaultCollectionRowComponent { } } - @HostListener("click") - protected click() { - // FIXME: Verify that this floating promise is intentional. If it is, add an explanatory comment and ensure there is proper error handling. - // eslint-disable-next-line @typescript-eslint/no-floating-promises - this.router.navigate([], { - queryParams: { collectionId: this.collection.id }, - queryParamsHandling: "merge", - }); - } - protected edit() { this.onEvent.next({ type: "editCollection", item: this.collection }); }