@jbrowse/product-core 5.0.0-beta.1 → 5.0.0-beta.11

package/esm/Session/Preferences.js

@@ -3,6 +3,7 @@ import { freezeDeep } from '@jbrowse/core/util/freezeDeep';
 import { addDisposer, types } from '@jbrowse/mobx-state-tree';
 import { autorun } from 'mobx';
 import { BaseSessionModel } from './BaseSession.js';
+import { applyDeveloperMode } from './developerMode.js';
 const PREFS_KEY = 'jbrowsePreferences';
 function loadStoredPreferences() {
     const stored = localStorageGetItem(PREFS_KEY);
@@ -34,8 +35,7 @@ export function PreferencesSessionMixin(pluginManager) {
         .actions(self => ({
         afterAttach() {
             // the restore path bypasses `setPreferenceOverride`, so it freezes here
-            // too — a promoted default read back from localStorage is shared by
-            // reference exactly like a freshly set one
+            // too
             self.preferencesOverrides.replace(freezeDeep(loadStoredPreferences()));
             // Applied once, here, rather than reactively: the same setting has to
             // hold in the RPC workers (which format tooltip strings from jexl
@@ -43,6 +43,10 @@ export function PreferencesSessionMixin(pluginManager) {
             // main-thread update would leave the app formatted two ways at once.
             // The dialog says a reload is needed; this is where the reload lands.
             setNumberGrouping(self.numberGrouping);
+            // and the same second pass for the developer-notice channel, whose
+            // admin default BaseSession already read: a user's stored override is
+            // only loaded a line above this
+            applyDeveloperMode(self);
             addDisposer(self, autorun(function persistPreferences() {
                 // observable.map doesn't JSON-serialize to an object on its own;
                 // fromEntries also reads every key so the autorun re-persists on