diff options
| author | かっこかり <67428053+kakkokari-gtyih@users.noreply.github.com> | 2025-11-24 16:52:46 +0900 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2025-11-24 16:52:46 +0900 |
| commit | 2ee04860fb7cdd09801dc96bf2bc561c4de00d9b (patch) | |
| tree | 27e8bba4af7b4277c327bc9193963e4cd95ad28a /packages/frontend/src/preferences | |
| parent | Bump version to 2025.11.1-alpha.0 (diff) | |
| download | misskey-2ee04860fb7cdd09801dc96bf2bc561c4de00d9b.tar.gz misskey-2ee04860fb7cdd09801dc96bf2bc561c4de00d9b.tar.bz2 misskey-2ee04860fb7cdd09801dc96bf2bc561c4de00d9b.zip | |
enhance(frontend): preferenceのタブ間同期にBroadcast Channelを使用するように (#16819)
* enhance(frontend): preferenceのタブ間同期にBroadcast Channelを使用するように
* fix
* refactor: EventEmitterをextendする形に変更
Diffstat (limited to 'packages/frontend/src/preferences')
| -rw-r--r-- | packages/frontend/src/preferences/manager.ts | 19 |
1 files changed, 18 insertions, 1 deletions
diff --git a/packages/frontend/src/preferences/manager.ts b/packages/frontend/src/preferences/manager.ts index b6d3d55a5f..5949ee71eb 100644 --- a/packages/frontend/src/preferences/manager.ts +++ b/packages/frontend/src/preferences/manager.ts @@ -4,6 +4,7 @@ */ import { computed, onUnmounted, ref, watch } from 'vue'; +import { EventEmitter } from 'eventemitter3'; import { host, version } from '@@/js/config.js'; import { PREF_DEF } from './def.js'; import type { Ref, WritableComputedRef } from 'vue'; @@ -100,6 +101,14 @@ type PreferencesDefinitionRecord<Default, T = Default extends (...args: any) => export type PreferencesDefinition = Record<string, PreferencesDefinitionRecord<any>>; +type PreferencesManagerEvents = { + 'committed': <K extends keyof PREF>(ctx: { + key: K; + value: ValueOf<K>; + oldValue: ValueOf<K>; + }) => void; +}; + export function definePreferences<T extends Record<string, unknown>>(x: { [K in keyof T]: PreferencesDefinitionRecord<T[K]> }): { @@ -180,7 +189,7 @@ function normalizePreferences(preferences: PossiblyNonNormalizedPreferencesProfi // TODO: PreferencesManagerForGuest のような非ログイン専用のクラスを分離すればthis.currentAccountのnullチェックやaccountがnullであるスコープのレコード挿入などが不要になり綺麗になるかもしれない // と思ったけど操作アカウントが存在しない場合も考慮する現在の設計の方が汎用的かつ堅牢かもしれない // NOTE: accountDependentな設定は初期状態であってもアカウントごとのスコープでレコードを作成しておかないと、サーバー同期する際に正しく動作しなくなる -export class PreferencesManager { +export class PreferencesManager extends EventEmitter<PreferencesManagerEvents> { private io: StorageProvider; private currentAccount: { id: string } | null; public profile: PreferencesProfile; @@ -201,6 +210,8 @@ export class PreferencesManager { }; constructor(io: StorageProvider, currentAccount: { id: string } | null) { + super(); + this.io = io; this.currentAccount = currentAccount; @@ -246,6 +257,12 @@ export class PreferencesManager { this.rewriteRawState(key, v); + this.emit('committed', { + key, + value: v, + oldValue: this.s[key], + }); + const record = this.getMatchedRecordOf(key); if (parseScope(record[0]).account == null && isAccountDependentKey(key) && currentAccount != null) { |