Refactor prepare/redraw for consistency

b89d41a70145bcf1ca6dc47d3dd00a21667b3ffe

Wolfsblvt <wolfsblvt@gmail.com>

public/scripts/extensions/quick-reply/src/ui/SettingsUi.js+12 -10 Viewed

             this.currentQrSet.name = newName;
             await this.currentQrSet.save();
             // Update the optionit in allboth selectset dropdownslists
 -            /** @type {NodeListOf<HTMLOptionElement>} */
 -            const options = this.dom.querySelectorAll(`#qr--set option[value="${oldName}"], select.qr--set option[value="${oldName}"]`);
 -            options.forEach(option => {
 -                option.value = newName;
 -                option.textContent = newName;
 -            });
+-
 -            // Update in in both set lists
             this.settings.config.setList.forEach(set => {
                 if (set.set.name === oldName) {
                     set.set.name = newName;

                     set.set.name = newName;
+                }
             });
+-
             this.settings.save();
++
 +            // Update the option in the current selected QR dropdown. All others will be refreshed via the prepare calls below.
 +            /** @type {HTMLOptionElement} */
 +            const option = this.currentSet.querySelector(`#qr--set option[value="${oldName}"]`);
 +            option.value = newName;
 +            option.textContent = newName;
++
             this.currentSet.value = newName;
 +            this.onQrSetChange();
 +            this.prepareGlobalSetList();
 +            this.prepareChatSetList();
++
             console.info(`Quick Reply Set renamed from ""${oldName}" to "${newName}".`);
+        }
+    }