v1.0.28: fix name dropdown not restoring when switching buttons

onDidReceiveGlobalSettings fired before the users list was populated,
so the saved userId couldn't be selected. Now stored in globalUserId
variable and applied when populateUsers runs, regardless of order.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
This commit is contained in:
pdmarf
2026-04-24 08:50:39 +01:00
co-authored by Claude Sonnet 4.6
parent 5426a388ba
commit c3add8da27
7 changed files with 12 additions and 10 deletions
@@ -161,6 +161,7 @@
var currentSettings = {};
var credSaveTimer = null;
var credConfigured = false;
var globalUserId = "";
function setStatus(msg, cls) {
var el = document.getElementById("statusText");
@@ -264,10 +265,11 @@
$PI.onDidReceiveGlobalSettings(function(jsn) {
var s = jsn.payload.settings || {};
document.getElementById("notionToken").value = s.notionToken || "";
if (s.userId) {
globalUserId = s.userId || "";
if (globalUserId) {
var sel = document.getElementById("userId");
if (sel.querySelector('option[value="' + s.userId + '"]')) {
sel.value = s.userId;
if (sel.querySelector('option[value="' + globalUserId + '"]')) {
sel.value = globalUserId;
}
}
@@ -306,8 +308,7 @@
document.getElementById("versionText").textContent = "v" + payload.version;
}
if (payload.users) {
var savedUserId = document.getElementById("userId").value;
populateUsers(payload.users, savedUserId);
populateUsers(payload.users, globalUserId);
}
if (payload.error) {
setStatus(payload.error, "error");