diff --git a/apps/browser/src/vault/popup/services/vault-popup-items.service.spec.ts b/apps/browser/src/vault/popup/services/vault-popup-items.service.spec.ts index 1900d35d9d9..5b0eb63998d 100644 --- a/apps/browser/src/vault/popup/services/vault-popup-items.service.spec.ts +++ b/apps/browser/src/vault/popup/services/vault-popup-items.service.spec.ts @@ -262,13 +262,6 @@ describe("VaultPopupItemsService", () => { }); }); - it("should sort by last used then by name", (done) => { - service.favoriteCiphers$.subscribe((ciphers) => { - expect(cipherServiceMock.sortCiphersByLastUsedThenName).toHaveBeenCalled(); - done(); - }); - }); - it("should filter favoriteCiphers$ down to search term", (done) => { const cipherList = Object.values(allCiphers); const searchText = "Card 2"; diff --git a/apps/browser/src/vault/popup/services/vault-popup-items.service.ts b/apps/browser/src/vault/popup/services/vault-popup-items.service.ts index b093fa7131c..93aa8cdaba9 100644 --- a/apps/browser/src/vault/popup/services/vault-popup-items.service.ts +++ b/apps/browser/src/vault/popup/services/vault-popup-items.service.ts @@ -164,16 +164,13 @@ export class VaultPopupItemsService { /** * List of favorite ciphers that are not currently suggested for autofill. - * Ciphers are sorted by last used date, then by name. + * Ciphers are sorted by name. */ favoriteCiphers$: Observable = this.autoFillCiphers$.pipe( withLatestFrom(this._filteredCipherList$), map(([autoFillCiphers, ciphers]) => ciphers.filter((cipher) => cipher.favorite && !autoFillCiphers.includes(cipher)), ), - map((ciphers) => - ciphers.sort((a, b) => this.cipherService.sortCiphersByLastUsedThenName(a, b)), - ), shareReplay({ refCount: false, bufferSize: 1 }), );