fix: Toggled options don't cause state save

oliversalzburg · Apr 29, 2022 · c2c4f17 · c2c4f17
1 parent 7d47c23
commit c2c4f17
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/packages/userscript/source/ui/BonfireSettingsUi.ts b/packages/userscript/source/ui/BonfireSettingsUi.ts
@@ -328,11 +328,11 @@ export class BonfireSettingsUi extends SettingsSectionUi<BonfireSettings> {
   ): JQuery<HTMLElement> {
     const element = this._getOption(name, option, label, delimiter, {
       onCheck: () => {
-        option.enabled = true;
+        this._host.updateOptions(() => (option.enabled = true));
         this._host.imessage("status.auto.enable", [label]);
       },
       onUnCheck: () => {
-        option.enabled = false;
+        this._host.updateOptions(() => (option.enabled = false));
         this._host.imessage("status.auto.disable", [label]);
       },
     });

diff --git a/packages/userscript/source/ui/SpaceSettingsUi.ts b/packages/userscript/source/ui/SpaceSettingsUi.ts
@@ -238,11 +238,11 @@ export class SpaceSettingsUi extends SettingsSectionUi<SpaceSettings> {
   ): JQuery<HTMLElement> {
     const element = this._getOption(name, option, i18nName, delimiter, {
       onCheck: () => {
-        option.enabled = true;
+        this._host.updateOptions(() => (option.enabled = true));
         this._host.imessage("status.auto.enable", [i18nName]);
       },
       onUnCheck: () => {
-        option.enabled = false;
+        this._host.updateOptions(() => (option.enabled = false));
         this._host.imessage("status.auto.disable", [i18nName]);
       },
     });