Skip to content

Commit

Permalink
fix: Do not override multiple times the default autofill settings
Browse files Browse the repository at this point in the history
It may be linked to a bug where the default value is set after an
update.
  • Loading branch information
zatteo committed Dec 17, 2024
1 parent a1b7b34 commit 4512328
Showing 1 changed file with 4 additions and 1 deletion.
5 changes: 4 additions & 1 deletion apps/browser/src/background/runtime.background.ts
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@ import { firstValueFrom, map, mergeMap } from "rxjs";
import { LockService } from "@bitwarden/auth/common";
import { NotificationsService } from "@bitwarden/common/abstractions/notifications.service";
import { AccountService } from "@bitwarden/common/auth/abstractions/account.service";
import { AutofillOverlayVisibility, ExtensionCommand } from "@bitwarden/common/autofill/constants";
import { ExtensionCommand } from "@bitwarden/common/autofill/constants";
import { AutofillSettingsServiceAbstraction } from "@bitwarden/common/autofill/services/autofill-settings.service";
import { FeatureFlag } from "@bitwarden/common/enums/feature-flag.enum";
import { ProcessReloadServiceAbstraction } from "@bitwarden/common/key-management/abstractions/process-reload.service";
Expand Down Expand Up @@ -433,9 +433,12 @@ export default class RuntimeBackground {
}
//*/

// Cozy customization, useless because OnFieldFocus is already our default value
/*
await this.autofillSettingsService.setInlineMenuVisibility(
AutofillOverlayVisibility.OnFieldFocus,
);
//*/

if (await this.environmentService.hasManagedEnvironment()) {
await this.environmentService.setUrlsToManagedEnvironment();
Expand Down

0 comments on commit 4512328

Please sign in to comment.