diff --git a/webui/src/lib/local-preferences.ts b/webui/src/lib/local-preferences.ts index 67f42a2e2..5405b1c0a 100644 --- a/webui/src/lib/local-preferences.ts +++ b/webui/src/lib/local-preferences.ts @@ -18,7 +18,7 @@ export const DEFAULT_LOCAL_PREFS: LocalPreferences = { density: "comfortable", activityMode: "auto", codeWrap: true, - brandLogos: false, + brandLogos: true, browserNotifications: false, fileEditDisplayMode: "summary", }; @@ -36,7 +36,7 @@ export function readLocalPreferences(): LocalPreferences { density: parsed.density === "compact" ? "compact" : "comfortable", activityMode: parsed.activityMode === "expanded" ? "expanded" : "auto", codeWrap: parsed.codeWrap !== false, - brandLogos: parsed.brandLogos === true, + brandLogos: parsed.brandLogos !== false, browserNotifications: parsed.browserNotifications === true, fileEditDisplayMode: normalizeFileEditDisplayMode(parsed.fileEditDisplayMode), }; diff --git a/webui/src/tests/local-preferences.test.ts b/webui/src/tests/local-preferences.test.ts index 5e207f194..6e17e672c 100644 --- a/webui/src/tests/local-preferences.test.ts +++ b/webui/src/tests/local-preferences.test.ts @@ -2,6 +2,7 @@ import { beforeEach, describe, expect, it } from "vitest"; import { DEFAULT_LOCAL_PREFS, + LOCAL_PREFS_STORAGE_KEY, readLocalPreferences, writeLocalPreferences, } from "@/lib/local-preferences"; @@ -9,6 +10,17 @@ import { describe("local preferences", () => { beforeEach(() => localStorage.clear()); + it("shows third-party brand logos by default while preserving an explicit opt-out", () => { + expect(DEFAULT_LOCAL_PREFS.brandLogos).toBe(true); + expect(readLocalPreferences().brandLogos).toBe(true); + + localStorage.setItem( + LOCAL_PREFS_STORAGE_KEY, + JSON.stringify({ brandLogos: false }), + ); + expect(readLocalPreferences().brandLogos).toBe(false); + }); + it("keeps browser notifications opt-in", () => { expect(DEFAULT_LOCAL_PREFS.browserNotifications).toBe(false); expect(readLocalPreferences().browserNotifications).toBe(false);