.gitignore

@@ -26,4 +26,3 @@ scripts/langmover/lang2
 scripts/langmover/out
 tinyproxy.conf
 static/banner.svg
-start.sh

Makefile

@@ -196,7 +196,7 @@ STATIC_TARGET = $(STATIC_SRC:static/%=$(DATA)/web/%)
 COPY_SRC = images/banner.svg jfa-go.service LICENSE $(LANG_SRC) $(STATIC_SRC)
 COPY_TARGET = $(DATA)/jfa-go.service
 # $(DATA)/LICENSE $(LANG_TARGET) $(STATIC_TARGET) $(DATA)/web/css/$(CSSVERSION)bundle.css
-$(COPY_TARGET): $(INLINE_TARGET) $(STATIC_SRC) $(LANG_SRC)
+$(COPY_TARGET): $(INLINE_TARGET) $(STATIC_SRC)
 	$(info copying crash page)
 	cp $(DATA)/crash.html $(DATA)/html/
 	$(info copying static data)

config.go

@@ -127,8 +127,6 @@ func (app *appContext) loadConfig() error {
 	app.MustSetValue("user_expiry", "adjustment_email_html", "jfa-go:"+"expiry-adjusted.html")
 	app.MustSetValue("user_expiry", "adjustment_email_text", "jfa-go:"+"expiry-adjusted.txt")
 
-	app.MustSetValue("email", "collect", "true")
-
 	app.MustSetValue("matrix", "topic", "Jellyfin notifications")
 	app.MustSetValue("matrix", "show_on_reg", "true")
 

config/config-base.json

@@ -896,20 +896,11 @@
                     "value": false,
                     "description": "Send emails as plain text instead of HTML."
                 },
-                "collect": {
-                    "name": "Collect on sign-up",
-                    "required": false,
-                    "requires_restart": false,
-                    "depends_true": "method",
-                    "type": "bool",
-                    "value": true,
-                    "description": "Ask for an email address on the sign-up form."
-                },
                 "required": {
                     "name": "Require on sign-up",
                     "required": false,
                     "requires_restart": false,
-                    "depends_true": "collect",
+                    "depends_true": "method",
                     "type": "bool",
                     "value": false,
                     "description": "Require an email address on sign-up."
@@ -918,7 +909,6 @@
                     "name": "Require unique address",
                     "required": false,
                     "requires_restart": true,
-                    "depends_true": "method",
                     "type": "bool",
                     "value": false,
                     "description": "Disables using the same address on multiple accounts."

html/admin.html

@@ -312,27 +312,25 @@
             <form class="relative mx-auto my-[10%] w-4/5 lg:w-2/3 content card" id="form-editor" href="">
                 <span class="heading"><span id="header-editor"></span> <span class="modal-close">&times;</span></span>
                 <div class="row">
-                    <div class="col card ~neutral @low flex flex-col gap-2 justify-between">
+                    <div class="col card ~neutral @low">
-                        <div class="flex flex-col gap-2">
+                        <aside class="aside sm ~urge dark:~d_info mb-2 @low" id="aside-editor"></aside> 
-                            <aside class="aside sm ~urge dark:~d_info @low" id="aside-editor"></aside> 
+                        <span class="label supra" for="editor-variables" id="label-editor-variables">{{ .strings.variables }}</span>
-                            <label class="label supra" for="editor-variables" id="label-editor-variables">{{ .strings.variables }}</label>
+                        <div id="editor-variables" class="mt-4"></div>
-                            <div id="editor-variables" class="flex flex-row gap-2 flex-wrap"></div>
                         <span class="label supra" for="editor-conditionals" id="label-editor-conditionals">{{ .strings.conditionals }}</span>
                         <div id="editor-conditionals"></div>
                         <label class="label supra" for="textarea-editor">{{ .strings.message }}</label>
-                            <textarea id="textarea-editor" class="textarea full-width flex-auto ~neutral @low font-mono"></textarea>
+                        <textarea id="textarea-editor" class="textarea full-width flex-auto ~neutral @low mt-4 font-mono"></textarea>
-                        </div>
+                        <p class="support mt-4 mb-2">{{ .strings.markdownSupported }}</p>
-                        <div class="flex flex-col gap-2">
+                        <div class="flex-row">
-                            <p class="support">{{ .strings.markdownSupported }}</p>
+                            <label class="full-width ml-2">
-                            <label class="w-full">
                                 <input type="submit" class="unfocused">
-                                <span class="button ~urge @low w-full supra submit">{{ .strings.submit }}</span>
+                                <span class="button ~urge @low full-width center supra submit">{{ .strings.submit }}</span>
                             </label>
                         </div>
                     </div>
-                    <div class="col card ~neutral @low flex flex-col gap-2">
+                    <div class="col card ~neutral @low">
                         <span class="subheading supra">{{ .strings.preview }}</span>
-                        <div id="editor-preview"></div>
+                        <div class="mt-8" id="editor-preview"></div>
                     </div>
                 </div>
             </form>

lang/common/en-us.json

@@ -48,8 +48,7 @@
         "errorConnection": "Couldn't connect to jfa-go.",
         "errorUnknown": "Unknown error.",
         "error401Unauthorized": "Unauthorized. Try refreshing the page.",
-        "errorSaveSettings": "Couldn't save settings.",
+        "errorSaveSettings": "Couldn't save settings."
-        "errorSpecialSymbols": "Field cannot contain special symbols."
     },
     "quantityStrings": {
         "year": {

logger/logger.go

@@ -52,14 +52,6 @@ func lshortfile() string {
 	return Lshortfile(3)
 }
 
-func LshortfileTree() string {
-	out := ""
-	for i := 6; i >= 0; i-- {
-		out += strconv.Itoa(i) + ":" + Lshortfile(i) + " "
-	}
-	return out
-}
-
 func NewLogger(out io.Writer, prefix string, flag int, color c.Attribute) (l *Logger) {
 	l = &Logger{}
 	// Use reimplemented Lshortfile since wrapping the log functions messes them up
@@ -104,13 +96,6 @@ func (l *Logger) PrintfCustomLevel(level int, format string, v ...interface{}) {
 	l.logger.Print(out)
 }
 
-func (l *Logger) PrintfNoFile(format string, v ...interface{}) {
-	if l.empty {
-		return
-	}
-	l.logger.Print(l.printer.Sprintf(format, v...))
-}
-
 func (l *Logger) Print(v ...interface{}) {
 	if l.empty {
 		return

logmessages/logmessages.go

@@ -175,7 +175,6 @@ const (
 	AccountLinked              = "account already linked and require_unique enabled"
 	AccountUnverified          = "unverified"
 	FailedSetDiscordMemberRole = "Failed to apply/remove " + Discord + " member role: %v"
-	InvalidChar                = "Invalid character '%c'"
 
 	FailedSetEmailAddress = "Failed to set email address for %s user \"%s\": %v"
 

models.go

@@ -6,8 +6,7 @@ import (
 
 type stringResponse struct {
 	Response string `json:"response" example:"message"`
-	ErrorText string `json:"error" example:"No special symbols allowed."`
+	Error    string `json:"error" example:"errorDescription"`
-	ErrorCode string `json:"error_code" example:"errorSpecialSymbols"`
 }
 
 type boolResponse struct {

ts/form.ts

@@ -176,32 +176,13 @@ const rePasswordField = document.getElementById("create-reenter-password") as HT
 
 let captcha = new Captcha(window.code, window.captcha, window.reCAPTCHA, false);
 
-const clearSubmitButton = () => {
-    submitInput.setCustomValidity("");
-    submitSpan.title = "";
-};
-
-const invalidMessage = (el: HTMLInputElement, msg: string) => {
-    el.setCustomValidity(msg);
-    submitInput.setCustomValidity(msg);
-    submitSpan.title = msg;
-};
-
 function _baseValidator(oncomplete: (valid: boolean) => void, captchaValid: boolean): void {
-    clearSubmitButton();
     if (window.emailRequired) {
         if (!emailField.value.includes("@")) {
             oncomplete(false);
             return;
         }
     }
-    usernameField.setCustomValidity("");
-    // Jellyfin doesn't like having "+" in the username field
-    if (usernameField.value.includes("+")) {
-        invalidMessage(usernameField, window.messages["errorSpecialSymbols"]);
-        oncomplete(false);
-        return;
-    }
     if (window.discordEnabled && window.discordRequired && !discordVerified) {
         oncomplete(false);
         return;

ts/modules/accounts.ts

@@ -1138,12 +1138,7 @@ export class accountsList {
                         console.log("User created, but welcome email failed");
                     }
                 } else {
-                    let msg = window.lang.var("notifications", "errorUserCreated", `"${send['username']}"`);
+                    window.notifications.customError("addUser", window.lang.var("notifications", "errorUserCreated", `"${send['username']}"`));
-                    if ("error" in req.response) {
-                        let realError = window.lang.notif(req.response["error"]);
-                        if (realError) msg = realError;
-                    }
-                    window.notifications.customError("addUser", msg);
                 }
                 if (req.response["error"] as String) {
                     console.log(req.response["error"]);

ts/modules/settings.ts

@@ -2,12 +2,6 @@ import { _get, _post, _delete, _download, _upload, toggleLoader, addLoader, remo
 import { Marked } from "@ts-stack/markdown";
 import { stripMarkdown } from "../modules/stripmd.js";
 
-const toBool = (s: string): boolean => {
-    let b = Boolean(s);
-    if (s == "false") b = false;
-    return b;
-}
-
 interface BackupDTO {
     size: string;
     name: string;
@@ -15,8 +9,8 @@ interface BackupDTO {
     date: number;
 }
 
-interface settingsChangedEvent extends Event { 
+interface settingsBoolEvent extends Event { 
-    detail: string;
+    detail: boolean;
 }
 
 interface Meta {
@@ -58,8 +52,7 @@ const splitDependant = (section: string, dep: string): string[] => {
     return parts
 };
 
-class DOMSetting {
+class DOMInput {
-    protected _hideEl: HTMLElement;
     protected _input: HTMLInputElement;
     protected _container: HTMLDivElement;
     protected _tooltip: HTMLDivElement;
@@ -69,20 +62,11 @@ class DOMSetting {
     protected _section: string;
     protected _name: string;
 
-    hide = () => {
+    hide = () => this._input.parentElement.classList.add("unfocused");
-        this._hideEl.classList.add("unfocused");
+    show = () => this._input.parentElement.classList.remove("unfocused");
-        const event = new CustomEvent(`settings-${this._section}-${this._name}`, { "detail": false })
-        document.dispatchEvent(event);
 
-    };
+    private _advancedListener = (event: settingsBoolEvent) => {
-    show = () => {
+        if (!Boolean(event.detail)) {
-        this._hideEl.classList.remove("unfocused");
-        const event = new CustomEvent(`settings-${this._section}-${this._name}`, { "detail": this.valueAsString() })
-        document.dispatchEvent(event);
-    };
-
-    private _advancedListener = (event: settingsChangedEvent) => {
-        if (!toBool(event.detail)) {
             this.hide();
         } else {
             this.show();
@@ -116,11 +100,9 @@ class DOMSetting {
     get required(): boolean { return this._required.classList.contains("badge"); }
     set required(state: boolean) {
         if (state) {
-            this._required.classList.remove("unfocused");
             this._required.classList.add("badge", "~critical");
             this._required.textContent = "*";
         } else {
-            this._required.classList.add("unfocused");
             this._required.classList.remove("badge", "~critical");
             this._required.textContent = "";
         }
@@ -129,11 +111,9 @@ class DOMSetting {
     get requires_restart(): boolean { return this._restart.classList.contains("badge"); }
     set requires_restart(state: boolean) {
         if (state) {
-            this._restart.classList.remove("unfocused");
             this._restart.classList.add("badge", "~info", "dark:~d_warning");
             this._restart.textContent = "R";
         } else {
-            this._restart.classList.add("unfocused");
             this._restart.classList.remove("badge", "~info", "dark:~d_warning");
             this._restart.textContent = "";
         }
@@ -143,44 +123,39 @@ class DOMSetting {
 
     onValueChange = () => {
         const event = new CustomEvent(`settings-${this._section}-${this._name}`, { "detail": this.valueAsString() })
-        const setEvent = new CustomEvent(`settings-set-${this._section}-${this._name}`, { "detail": this.valueAsString() })
         document.dispatchEvent(event);
-        document.dispatchEvent(setEvent);
         if (this.requires_restart) { document.dispatchEvent(new CustomEvent("settings-requires-restart")); }
     };
 
-    constructor(input: string, setting: Setting, section: string, name: string, inputOnTop: boolean = false) {
+    constructor(inputType: string, setting: Setting, section: string, name: string, customInput?: string) {
         this._section = section;
         this._name = name;
         this._container = document.createElement("div");
         this._container.classList.add("setting");
-        this._container.setAttribute("data-name", name);
+        this._container.setAttribute("data-name", name)
+        const defaultInput = `
+            <input type="${inputType}" class="input setting-input ~neutral @low mt-2 mb-2">
+        `;
         this._container.innerHTML = `
-        <label class="label flex flex-col gap-2">
+        <label class="label">
-            ${inputOnTop ? input : ""}
+            <span class="setting-label"></span> <span class="setting-required"></span> <span class="setting-restart"></span>
-            <div class="flex flex-row gap-2 items-baseline">
-                <span class="setting-label"></span>
             <div class="setting-tooltip tooltip right unfocused">
-                    <i class="icon ri-information-line align-baseline"></i>
+                <i class="icon ri-information-line"></i>
                 <span class="content sm"></span>
             </div>
-                <span class="setting-required unfocused"></span>
+            ${customInput ? customInput : defaultInput}
-                <span class="setting-restart unfocused"></span>
-            </div>
-            ${inputOnTop ? "" : input}
         </label>
         `;
         this._tooltip = this._container.querySelector("div.setting-tooltip") as HTMLDivElement;
         this._required = this._container.querySelector("span.setting-required") as HTMLSpanElement;
         this._restart = this._container.querySelector("span.setting-restart") as HTMLSpanElement;
-        // "input" variable should supply the HTML of an element with class "setting-input"
         this._input = this._container.querySelector(".setting-input") as HTMLInputElement;
         if (setting.depends_false || setting.depends_true) {
             let dependant = splitDependant(section, setting.depends_true || setting.depends_false);
             let state = true;
             if (setting.depends_false) { state = false; }
-            document.addEventListener(`settings-${dependant[0]}-${dependant[1]}`, (event: settingsChangedEvent) => {
+            document.addEventListener(`settings-${dependant[0]}-${dependant[1]}`, (event: settingsBoolEvent) => {
-                if (toBool(event.detail) !== state) {
+                if (Boolean(event.detail) !== state) {
                     this.hide();
                 } else {
                     this.show();
@@ -188,14 +163,13 @@ class DOMSetting {
             });
         }
         this._input.onchange = this.onValueChange;
-        document.addEventListener(`settings-loaded`, this.onValueChange);
+        this.update(setting);
-        this._hideEl = this._container;
     }
 
     get value(): any { return this._input.value; }
     set value(v: any) { this._input.value = v; }
 
-    update(s: Setting) {
+    update = (s: Setting) => {
         this.name = s.name;
         this.description = s.description;
         this.required = s.required;
@@ -207,17 +181,6 @@ class DOMSetting {
     asElement = (): HTMLDivElement => { return this._container; }
 }
 
-class DOMInput extends DOMSetting {
-    constructor(inputType: string, setting: Setting, section: string, name: string) {
-        super(
-            `<input type="${inputType}" class="input setting-input ~neutral @low">`,
-            setting, section, name,
-        );
-        // this._hideEl = this._input.parentElement;
-        this.update(setting);
-    }
-}
-
 interface SText extends Setting {
     value: string;
 }
@@ -228,6 +191,85 @@ class DOMText extends DOMInput implements SText {
     set value(v: string) { this._input.value = v; }
 }
 
+interface SList extends Setting {
+    value: string[];
+}
+
+class DOMList extends DOMInput implements SList {
+    protected _inputs: HTMLDivElement;
+    type: string = "list";
+    
+    valueAsString = (): string => { return this.value.join("|"); };
+
+    get value(): string[] {
+        let values = [];
+        const inputs = this._input.querySelectorAll("input") as NodeListOf<HTMLInputElement>;
+        for (let i in inputs) {
+            if (inputs[i].value) values.push(inputs[i].value);
+        }
+        return values;
+    }
+    set value(v: string[]) {
+        this._input.textContent = ``;
+        for (let val of v) {
+            let input = this.inputRow(val);
+            this._input.appendChild(input);
+        }
+        const addDummy = () => {
+            const dummyRow = this.inputRow();
+            const input = dummyRow.querySelector("input") as HTMLInputElement;
+            input.placeholder = window.lang.strings("add");
+            const onDummyChange = () => {
+                if (!(input.value)) return;
+                addDummy();
+                input.removeEventListener("change", onDummyChange);
+                input.placeholder = ``;
+            }
+            input.addEventListener("change", onDummyChange);
+            this._input.appendChild(dummyRow);
+        };
+        addDummy();
+    }
+
+    private inputRow(v: string = ""): HTMLDivElement {
+        let container = document.createElement("div") as HTMLDivElement;
+        container.classList.add("flex", "flex-row", "justify-between");
+        container.innerHTML = `
+            <input type="text" class="input ~neutral @low">
+            <button class="button ~neutral @low center -ml-10 rounded-s-none aria-label="${window.lang.strings("delete")}" title="${window.lang.strings("delete")}">
+                <i class="ri-close-line"></i>
+            </button>
+        `;
+        const input = container.querySelector("input") as HTMLInputElement;
+        input.value = v;
+        input.onchange = this.onValueChange;
+        const removeRow = container.querySelector("button") as HTMLButtonElement;
+        removeRow.onclick = () => {
+            if (!(container.nextElementSibling)) return;
+            container.remove();
+            this.onValueChange();
+        }
+        return container;
+    }
+
+    update = (s: Setting) => {
+        this.name = s.name;
+        this.description = s.description;
+        this.required = s.required;
+        this.requires_restart = s.requires_restart;
+        this.value = s.value as string[];
+        this.advanced = s.advanced;
+    }
+    
+    asElement = (): HTMLDivElement => { return this._container; }
+
+    constructor(setting: Setting, section: string, name: string) {
+        super("list", setting, section, name,
+            `<div class="setting-input flex flex-col gap-2 mt-2 mb-2"></div>`
+        );
+    }
+}
+
 interface SPassword extends Setting {
     value: string;
 }
@@ -258,107 +300,209 @@ class DOMNumber extends DOMInput implements SNumber {
     set value(v: number) { this._input.value = ""+v; }
 }
 
-interface SList extends Setting {
-    value: string[];
-}
-class DOMList extends DOMSetting implements SList {
-    protected _inputs: HTMLDivElement;
-    type: string = "list";
-    
-    valueAsString = (): string => { return this.value.join("|"); };
-
-    get value(): string[] {
-        let values = [];
-        const inputs = this._input.querySelectorAll("input") as NodeListOf<HTMLInputElement>;
-        for (let i in inputs) {
-            if (inputs[i].value) values.push(inputs[i].value);
-        }
-        return values;
-    }
-    set value(v: string[]) {
-        this._input.textContent = ``;
-        for (let val of v) {
-            let input = this.inputRow(val);
-            this._input.appendChild(input);
-        }
-        const addDummy = () => {
-            const dummyRow = this.inputRow();
-            const input = dummyRow.querySelector("input") as HTMLInputElement;
-            input.placeholder = window.lang.strings("add");
-            const onDummyChange = () => {
-                if (!(input.value)) return;
-                addDummy();
-                input.removeEventListener("change", onDummyChange);
-                input.removeEventListener("keyup", onDummyChange);
-                input.placeholder = ``;
-            }
-            input.addEventListener("change", onDummyChange);
-            input.addEventListener("keyup", onDummyChange);
-            this._input.appendChild(dummyRow);
-        };
-        addDummy();
-    }
-
-    private inputRow(v: string = ""): HTMLDivElement {
-        let container = document.createElement("div") as HTMLDivElement;
-        container.classList.add("flex", "flex-row", "justify-between");
-        container.innerHTML = `
-            <input type="text" class="input ~neutral @low">
-            <button class="button ~neutral @low center -ml-10 rounded-s-none aria-label="${window.lang.strings("delete")}" title="${window.lang.strings("delete")}">
-                <i class="ri-close-line"></i>
-            </button>
-        `;
-        const input = container.querySelector("input") as HTMLInputElement;
-        input.value = v;
-        input.onchange = this.onValueChange;
-        const removeRow = container.querySelector("button") as HTMLButtonElement;
-        removeRow.onclick = () => {
-            if (!(container.nextElementSibling)) return;
-            container.remove();
-            this.onValueChange();
-        }
-        return container;
-    }
-    
-    constructor(setting: Setting, section: string, name: string) {
-        super(
-            `<div class="setting-input flex flex-col gap-2"></div>`,
-            setting, section, name,
-        );
-        // this._hideEl = this._input.parentElement;
-        this.update(setting);
-    }
-}
-
 interface SBool extends Setting {
     value: boolean;
 }
-class DOMBool extends DOMSetting implements SBool {
+class DOMBool implements SBool {
+    protected _input: HTMLInputElement;
+    private _container: HTMLDivElement;
+    private _tooltip: HTMLDivElement;
+    private _required: HTMLSpanElement;
+    private _restart: HTMLSpanElement;
     type: string = "bool";
+    private _advanced: boolean;
+    
+    hide = () => this._input.parentElement.classList.add("unfocused");
+    show = () => this._input.parentElement.classList.remove("unfocused");
+
+    private _advancedListener = (event: settingsBoolEvent) => {
+        if (!Boolean(event.detail)) {
+            this.hide();
+        } else {
+            this.show();
+        }
+    }
+
+    get advanced(): boolean { return this._advanced; }
+    set advanced(advanced: boolean) {
+        this._advanced = advanced;
+        if (advanced) {
+            document.addEventListener("settings-advancedState", this._advancedListener);
+        } else {
+            document.removeEventListener("settings-advancedState", this._advancedListener);
+        }
+    }
+
+    get name(): string { return this._container.querySelector("span.setting-label").textContent; }
+    set name(n: string) { this._container.querySelector("span.setting-label").textContent = n; }
+
+    get description(): string { return this._tooltip.querySelector("span.content").textContent; } 
+    set description(d: string) {
+        const content = this._tooltip.querySelector("span.content") as HTMLSpanElement;
+        content.textContent = d;
+        if (d == "") {
+            this._tooltip.classList.add("unfocused");
+        } else {
+            this._tooltip.classList.remove("unfocused");
+        }
+    }
+
+    get required(): boolean { return this._required.classList.contains("badge"); }
+    set required(state: boolean) {
+        if (state) {
+            this._required.classList.add("badge", "~critical");
+            this._required.textContent = "*";
+        } else {
+            this._required.classList.remove("badge", "~critical");
+            this._required.textContent = "";
+        }
+    }
+    
+    get requires_restart(): boolean { return this._restart.classList.contains("badge"); }
+    set requires_restart(state: boolean) {
+        if (state) {
+            this._restart.classList.add("badge", "~info", "dark:~d_warning");
+            this._restart.textContent = "R";
+        } else {
+            this._restart.classList.remove("badge", "~info", "dark:~d_warning");
+            this._restart.textContent = "";
+        }
+    }
+
+    valueAsString = (): string => { return ""+this.value; };
 
     get value(): boolean { return this._input.checked; }
     set value(state: boolean) { this._input.checked = state; }
     
     constructor(setting: SBool, section: string, name: string) {
-        super(
+        this._container = document.createElement("div");
-            `<input type="checkbox" class="setting-input">`,
+        this._container.classList.add("setting");
-            setting, section, name, true,
+        this._container.setAttribute("data-name", name)
-        );
+        this._container.innerHTML = `
-        const label = this._container.getElementsByTagName("LABEL")[0];
+        <label class="switch mb-2">
-        label.classList.remove("flex-col");
+            <input type="checkbox">
-        label.classList.add("flex-row");
+            <span class="setting-label"></span> <span class="setting-required"></span> <span class="setting-restart"></span>
-        // this._hideEl = this._input.parentElement;
+            <div class="setting-tooltip tooltip right unfocused">
+                <i class="icon ri-information-line"></i>
+                <span class="content sm"></span>
+            </div>
+        </label>
+        `;
+        this._tooltip = this._container.querySelector("div.setting-tooltip") as HTMLDivElement;
+        this._required = this._container.querySelector("span.setting-required") as HTMLSpanElement;
+        this._restart = this._container.querySelector("span.setting-restart") as HTMLSpanElement;
+        this._input = this._container.querySelector("input[type=checkbox]") as HTMLInputElement;
+        const onValueChange = () => {
+            const event = new CustomEvent(`settings-${section}-${name}`, { "detail": this.valueAsString() })
+            document.dispatchEvent(event);
+        };
+        this._input.onchange = () => { 
+            onValueChange();
+            if (this.requires_restart) { document.dispatchEvent(new CustomEvent("settings-requires-restart")); }
+        };
+        document.addEventListener(`settings-loaded`, onValueChange);
+
+        if (setting.depends_false || setting.depends_true) {
+            let dependant = splitDependant(section, setting.depends_true || setting.depends_false);
+            let state = true;
+            if (setting.depends_false) { state = false; }
+            document.addEventListener(`settings-${dependant[0]}-${dependant[1]}`, (event: settingsBoolEvent) => {
+                if (Boolean(event.detail) !== state) {
+                    this.hide();
+                } else {
+                    this.show();
+                }
+            });
+        }
         this.update(setting);
     }
+    update = (s: SBool) => {
+        this.name = s.name;
+        this.description = s.description;
+        this.required = s.required;
+        this.requires_restart = s.requires_restart;
+        this.value = s.value;
+        this.advanced = s.advanced;
+    }
+    
+    asElement = (): HTMLDivElement => { return this._container; }
 }
 
 interface SSelect extends Setting {
     options: string[][];
     value: string;
 }
-class DOMSelect extends DOMSetting implements SSelect {
+class DOMSelect implements SSelect {
-    type: string = "bool";
+    protected _select: HTMLSelectElement;
+    private _container: HTMLDivElement;
+    private _tooltip: HTMLDivElement;
+    private _required: HTMLSpanElement;
+    private _restart: HTMLSpanElement;
     private _options: string[][];
+    type: string = "bool";
+    private _advanced: boolean;
+
+    hide = () => this._container.classList.add("unfocused");
+    show = () => this._container.classList.remove("unfocused");
+
+    private _advancedListener = (event: settingsBoolEvent) => {
+        if (!Boolean(event.detail)) {
+            this.hide();
+        } else {
+            this.show();
+        }
+    }
+
+    get advanced(): boolean { return this._advanced; }
+    set advanced(advanced: boolean) {
+        this._advanced = advanced;
+        if (advanced) {
+            document.addEventListener("settings-advancedState", this._advancedListener);
+        } else {
+            document.removeEventListener("settings-advancedState", this._advancedListener);
+        }
+    }
+
+    get name(): string { return this._container.querySelector("span.setting-label").textContent; }
+    set name(n: string) { this._container.querySelector("span.setting-label").textContent = n; }
+
+    get description(): string { return this._tooltip.querySelector("span.content").textContent; } 
+    set description(d: string) {
+        const content = this._tooltip.querySelector("span.content") as HTMLSpanElement;
+        content.textContent = d;
+        if (d == "") {
+            this._tooltip.classList.add("unfocused");
+        } else {
+            this._tooltip.classList.remove("unfocused");
+        }
+    }
+
+    get required(): boolean { return this._required.classList.contains("badge"); }
+    set required(state: boolean) {
+        if (state) {
+            this._required.classList.add("badge", "~critical");
+            this._required.textContent = "*";
+        } else {
+            this._required.classList.remove("badge", "~critical");
+            this._required.textContent = "";
+        }
+    }
+    
+    get requires_restart(): boolean { return this._restart.classList.contains("badge"); }
+    set requires_restart(state: boolean) {
+        if (state) {
+            this._restart.classList.add("badge", "~info", "dark:~d_warning");
+            this._restart.textContent = "R";
+        } else {
+            this._restart.classList.remove("badge", "~info", "dark:~d_warning");
+            this._restart.textContent = "";
+        }
+    }
+
+    valueAsString = (): string => { return ""+this.value; };
+
+    get value(): string { return this._select.value; }
+    set value(v: string) { this._select.value = v; }
 
     get options(): string[][] { return this._options; }
     set options(opt: string[][]) {
@@ -367,47 +511,88 @@ class DOMSelect extends DOMSetting implements SSelect {
         for (let option of this._options) {
             innerHTML += `<option value="${option[0]}">${option[1]}</option>`;
         }
-        this._input.innerHTML = innerHTML;
+        this._select.innerHTML = innerHTML;
     }
 
-    update(s: SSelect) {
-        this.options = s.options;
-        super.update(s);
-    };
-
     constructor(setting: SSelect, section: string, name: string) {
-        super(
-            `<div class="select ~neutral @low">
-                <select class="setting-select setting-input"></select>
-            </div>`,
-            setting, section, name,
-        );
         this._options = [];
-        // this._hideEl = this._container;
+        this._container = document.createElement("div");
+        this._container.classList.add("setting");
+        this._container.setAttribute("data-name", name)
+        this._container.innerHTML = `
+        <label class="label">
+            <span class="setting-label"></span> <span class="setting-required"></span> <span class="setting-restart"></span>
+            <div class="setting-tooltip tooltip right unfocused">
+                <i class="icon ri-information-line"></i>
+                <span class="content sm"></span>
+            </div>
+            <div class="select ~neutral @low mt-2 mb-2">
+                <select class="settings-select"></select>
+            </div>
+        </label>
+        `;
+        this._tooltip = this._container.querySelector("div.setting-tooltip") as HTMLDivElement;
+        this._required = this._container.querySelector("span.setting-required") as HTMLSpanElement;
+        this._restart = this._container.querySelector("span.setting-restart") as HTMLSpanElement;
+        this._select = this._container.querySelector("select.settings-select") as HTMLSelectElement;
+        if (setting.depends_false || setting.depends_true) {
+            let dependant = splitDependant(section, setting.depends_true || setting.depends_false);
+            let state = true;
+            if (setting.depends_false) { state = false; }
+            document.addEventListener(`settings-${dependant[0]}-${dependant[1]}`, (event: settingsBoolEvent) => {
+                if (Boolean(event.detail) !== state) {
+                    this.hide();
+                } else {
+                    this.show();
+                }
+            });
+        }
+        const onValueChange = () => {
+            const event = new CustomEvent(`settings-${section}-${name}`, { "detail": this.valueAsString() })
+            document.dispatchEvent(event);
+            if (this.requires_restart) { document.dispatchEvent(new CustomEvent("settings-requires-restart")); }
+        };
+        this._select.onchange = onValueChange;
+        document.addEventListener(`settings-loaded`, onValueChange);
+
+        const message = document.getElementById("settings-message") as HTMLElement;
+        message.innerHTML = window.lang.var("strings",
+                                            "settingsRequiredOrRestartMessage",
+                                            `<span class="badge ~critical">*</span>`,
+                                            `<span class="badge ~info dark:~d_warning">R</span>`
+        );
+
         this.update(setting);
     }
+    update = (s: SSelect) => {
+        this.name = s.name;
+        this.description = s.description;
+        this.required = s.required;
+        this.requires_restart = s.requires_restart;
+        this.options = s.options;
+        this.value = s.value;
+    }
+
+    asElement = (): HTMLDivElement => { return this._container; }
 }
 
 interface SNote extends Setting {
     value: string;
     style?: string;
 }
-class DOMNote extends DOMSetting implements SNote {
+class DOMNote implements SNote {
-    private _nameEl: HTMLElement;
+    private _container: HTMLDivElement;
+    private _aside: HTMLElement;
+    private _name: HTMLElement;
     private _description: HTMLElement;
     type: string = "note";
     private _style: string;
 
-    // We're a note, no one depends on us so we don't need to broadcast a state change.
+    hide = () => this._container.classList.add("unfocused");
-    hide = () => {
+    show = () => this._container.classList.remove("unfocused");
-		this._container.classList.add("unfocused");
-	};
-    show = () => {
-		this._container.classList.remove("unfocused");
-	};
 
-    get name(): string { return this._nameEl.textContent; }
+    get name(): string { return this._name.textContent; }
-    set name(n: string) { this._nameEl.textContent = n; }
+    set name(n: string) { this._name.textContent = n; }
 
     get description(): string { return this._description.textContent; }
     set description(d: string) {
@@ -417,37 +602,51 @@ class DOMNote extends DOMSetting implements SNote {
     valueAsString = (): string => { return ""; };
 
     get value(): string { return ""; }
-    set value(_: string) { return; }
+    set value(v: string) { return; }
     
     get required(): boolean { return false; }
-    set required(_: boolean) { return; }
+    set required(v: boolean) { return; }
     
     get requires_restart(): boolean { return false; }
-    set requires_restart(_: boolean) { return; }
+    set requires_restart(v: boolean) { return; }
 
     get style(): string { return this._style; }
     set style(s: string) {
-        this._input.classList.remove("~" + this._style);
+        this._aside.classList.remove("~" + this._style);
         this._style = s;
-        this._input.classList.add("~" + this._style);
+        this._aside.classList.add("~" + this._style);
     }
     
     constructor(setting: SNote, section: string) {
-        super(
+        this._container = document.createElement("div");
-            `
+        this._container.classList.add("setting");
-            <aside class="aside setting-input">
+        this._container.innerHTML = `
+        <aside class="aside my-2">
             <span class="font-bold setting-name"></span>
             <span class="content setting-description">
         </aside>
-            `, setting, section, "",
+        `;
-        );
+        this._name = this._container.querySelector(".setting-name");
-        // this._hideEl = this._container;
-        this._nameEl = this._container.querySelector(".setting-name");
         this._description = this._container.querySelector(".setting-description");
+        this._aside = this._container.querySelector("aside");
+
+        if (setting.depends_false || setting.depends_true) {
+            let dependant = splitDependant(section, setting.depends_true || setting.depends_false);
+            let state = true;
+            if (setting.depends_false) { state = false; }
+            document.addEventListener(`settings-${dependant[0]}-${dependant[1]}`, (event: settingsBoolEvent) => {
+                if (Boolean(event.detail) !== state) {
+                    this.hide();
+                } else {
+                    this.show();
+                }
+            });
+        }
+
         this.update(setting);
     }
 
-    update(s: SNote) {
+    update = (s: SNote) => {
         this.name = s.name;
         this.description = s.description;
         this.style = ("style" in s && s.style) ? s.style : "info";
@@ -472,7 +671,7 @@ class sectionPanel {
         this._sectionName = sectionName;
         this._settings = {};
         this._section = document.createElement("div") as HTMLDivElement;
-        this._section.classList.add("settings-section", "unfocused", "flex", "flex-col", "gap-2");
+        this._section.classList.add("settings-section", "unfocused");
         this._section.setAttribute("data-section", sectionName);
         let innerHTML = `
         <div class="flex flex-row justify-between">
@@ -484,7 +683,7 @@ class sectionPanel {
 
         innerHTML += `
         </div>
-        <p class="support lg settings-section-description">${s.meta.description}</p>
+        <p class="support lg my-2 settings-section-description">${s.meta.description}</p>
         `;
         this._section.innerHTML = innerHTML;
 
@@ -525,9 +724,7 @@ class sectionPanel {
                 }
                 if (setting.type != "note") {
                     this.values[name] = ""+setting.value;
-                    // settings-section-name: Implies the setting changed or was shown/hidden.
+                    document.addEventListener(`settings-${this._sectionName}-${name}`, (event: CustomEvent) => {
-                    // settings-set-section-name: Implies the setting changed.
-                    document.addEventListener(`settings-set-${this._sectionName}-${name}`, (event: CustomEvent) => {
                         // const oldValue = this.values[name];
                         this.values[name] = event.detail;
                         document.dispatchEvent(new CustomEvent("settings-section-changed"));
@@ -594,8 +791,9 @@ export class settingsList {
             let dependant = splitDependant(name, s.meta.depends_true || s.meta.depends_false);
             let state = true;
             if (s.meta.depends_false) { state = false; }
-            document.addEventListener(`settings-${dependant[0]}-${dependant[1]}`, (event: settingsChangedEvent) => {
+            document.addEventListener(`settings-${dependant[0]}-${dependant[1]}`, (event: settingsBoolEvent) => {
-                if (toBool(event.detail) !== state) {
+
+                if (Boolean(event.detail) !== state) {
                     button.classList.add("unfocused");
                     document.dispatchEvent(new CustomEvent(`settings-${name}`, { detail: false }));
                 } else {
@@ -603,16 +801,16 @@ export class settingsList {
                     document.dispatchEvent(new CustomEvent(`settings-${name}`, { detail: true }));
                 }
             });
-            document.addEventListener(`settings-${dependant[0]}`, (event: settingsChangedEvent) => {
+            document.addEventListener(`settings-${dependant[0]}`, (event: settingsBoolEvent) => {
-                if (toBool(event.detail) !== state) {
+                if (Boolean(event.detail) !== state) {
                     button.classList.add("unfocused");
                     document.dispatchEvent(new CustomEvent(`settings-${name}`, { detail: false }));
                 }
             }); 
         }
         if (s.meta.advanced) {
-            document.addEventListener("settings-advancedState", (event: settingsChangedEvent) => {
+            document.addEventListener("settings-advancedState", (event: settingsBoolEvent) => {
-                if (!toBool(event.detail)) {
+                if (!Boolean(event.detail)) {
                     button.classList.add("unfocused");
                 } else {
                     button.classList.remove("unfocused");
@@ -804,14 +1002,6 @@ export class settingsList {
                 this._searchbox.oninput(null);
             };
         };
-
-        // What possessed me to put this in the DOMSelect constructor originally? like what????????
-        const message = document.getElementById("settings-message") as HTMLElement;
-        message.innerHTML = window.lang.var("strings",
-                                            "settingsRequiredOrRestartMessage",
-                                            `<span class="badge ~critical">*</span>`,
-                                            `<span class="badge ~info dark:~d_warning">R</span>`
-        );
     }
 
     private _addMatrix = () => {
@@ -1088,7 +1278,7 @@ class MessageEditor {
                 let innerHTML = '';
                 for (let i = 0; i < this._templ.variables.length; i++) {
                     let ci = i % colors.length;
-                    innerHTML += '<span class="button ~' + colors[ci] +' @low"></span>'
+                    innerHTML += '<span class="button ~' + colors[ci] +' @low mb-4" style="margin-left: 0.25rem; margin-right: 0.25rem;"></span>'
                 }
                 if (this._templ.variables.length == 0) {
                     this._variablesLabel.classList.add("unfocused");

users.go

@@ -1,13 +1,10 @@
 package main
 
 import (
-	"fmt"
-	"strings"
 	"sync"
 	"time"
 
 	"github.com/gin-gonic/gin"
-	"github.com/hrfee/jfa-go/logger"
 	lm "github.com/hrfee/jfa-go/logmessages"
 	"github.com/hrfee/mediabrowser"
 	"github.com/lithammer/shortuuid/v3"
@@ -56,11 +53,9 @@ type NewUserData struct {
 func (app *appContext) NewUserPostVerification(p NewUserParams) (out NewUserData, pendingTasks *sync.WaitGroup) {
 	pendingTasks = &sync.WaitGroup{}
 	// Some helper functions which will behave as our app.info/error/debug
-	// And make sure we capture the correct caller location.
 	deferLogInfo := func(s string, args ...any) {
-		loc := logger.Lshortfile(2)
 		out.Log = func() {
-			app.info.PrintfNoFile(loc+" "+s, args...)
+			app.info.Printf(s, args)
 		}
 	}
 	/* deferLogDebug := func(s string, args ...any) {
@@ -69,19 +64,11 @@ func (app *appContext) NewUserPostVerification(p NewUserParams) (out NewUserData
 		}
 	} */
 	deferLogError := func(s string, args ...any) {
-		loc := logger.Lshortfile(2)
 		out.Log = func() {
-			app.err.PrintfNoFile(loc+" "+s, args...)
+			app.err.Printf(s, args)
 		}
 	}
 
-	if strings.ContainsRune(p.Req.Username, '+') {
-		deferLogError(lm.FailedCreateUser, lm.Jellyfin, p.Req.Username, fmt.Sprintf(lm.InvalidChar, '+'))
-		out.Status = 400
-		out.Message = "errorSpecialSymbols"
-		return
-	}
-
 	existingUser, _ := app.jf.UserByName(p.Req.Username, false)
 	if existingUser.Name != "" {
 		out.Message = lm.UserExists

views.go

@@ -677,7 +677,7 @@ func (app *appContext) NewUserFromConfirmationKey(invite Invite, key string, lan
 		nu.Log()
 	}
 	if !nu.Created {
-		// respond(nu.Status, nu.Message, gc)
+		respond(nu.Status, nu.Message, gc)
 		fail()
 		return
 	}