From 0b98473e85ce31bf7a4beb80077d29f2002ee1e6 Mon Sep 17 00:00:00 2001
From: RMidhunSuresh <hi@midhun.dev>
Date: Thu, 26 May 2022 23:36:37 +0530
Subject: [PATCH] Render a radio button for default variants

---
 .../session/settings/SettingsViewModel.js     | 14 ++++++--
 .../web/ui/session/settings/SettingsView.js   | 34 +++++++++++++++++--
 2 files changed, 43 insertions(+), 5 deletions(-)

diff --git a/src/domain/session/settings/SettingsViewModel.js b/src/domain/session/settings/SettingsViewModel.js
index 5c89236f..bab72d3d 100644
--- a/src/domain/session/settings/SettingsViewModel.js
+++ b/src/domain/session/settings/SettingsViewModel.js
@@ -131,8 +131,8 @@ export class SettingsViewModel extends ViewModel {
         return this._formatBytes(this._estimate?.usage);
     }
 
-    get themes() {
-        return this.platform.themeLoader.themes;
+    get themeMapping() {
+        return this.platform.themeLoader.themeMapping;
     }
 
     get activeTheme() {
@@ -185,5 +185,15 @@ export class SettingsViewModel extends ViewModel {
             this.emitChange("pushNotifications.serverError");
         }
     }
+
+    themeOptionChanged(themeId) {
+        if (themeId) {
+            this.setTheme(themeId);
+        }
+        // if there's no themeId, then the theme is going to be set via radio-buttons
+        // emit so that radio-buttons become displayed/hidden
+        this.emitChange("themeOption");
+    }
+
 }
 
diff --git a/src/platform/web/ui/session/settings/SettingsView.js b/src/platform/web/ui/session/settings/SettingsView.js
index dd7bbc03..06f98577 100644
--- a/src/platform/web/ui/session/settings/SettingsView.js
+++ b/src/platform/web/ui/session/settings/SettingsView.js
@@ -142,9 +142,37 @@ export class SettingsView extends TemplateView {
     _themeOptions(t, vm) {
         const activeTheme = vm.activeTheme;
         const optionTags = [];
-        for (const name of vm.themes) {
-            optionTags.push(t.option({value: name, selected: name === activeTheme}, name));
+        let isDarkSelected = null, isLightSelected = null; 
+        for (const [name, details] of Object.entries(vm.themeMapping)) {
+            let isSelected = null;
+            if (details.id === activeTheme) {
+                isSelected = true;
+            }
+            else if (details.dark?.id === activeTheme) {
+                isSelected = true;
+                isDarkSelected = true;
+            }
+            else if (details.light?.id === activeTheme) {
+                isSelected = true;
+                isLightSelected = true;
+            }
+            optionTags.push( t.option({ value: details.id ?? "", selected: isSelected} , name));
         }
-        return t.select({onChange: (e) => vm.setTheme(e.target.value)}, optionTags);
+        const select = t.select({ onChange: (e) => vm.themeOptionChanged(e.target.value) }, optionTags);
+        const radioButtons = t.form({
+            className: { hidden: () => select.options[select.selectedIndex].value !== "" },
+            onChange: (e) => {
+                const selectedThemeName = select.options[select.selectedIndex].text;
+                const themeId = vm.themeMapping[selectedThemeName][e.target.value].id;
+                vm.themeOptionChanged(themeId);
+            }
+        },
+        [
+            t.input({ type: "radio", name: "radio-chooser", value: "dark", id: "dark", checked: isDarkSelected }),
+            t.label({for: "dark"}, "dark"),
+            t.input({ type: "radio", name: "radio-chooser", value: "light", id: "light", checked: isLightSelected }),
+            t.label({for: "light"}, "light"),
+        ]);
+        return t.div({ className: "theme-chooser" }, [select, radioButtons]);
     }
 }