Fix TypeError when loading UI with custom palette selected (#5205)
* Add null check to fix circular dependency with custom palettes Co-authored-by: DedeHai <6280424+DedeHai@users.noreply.github.com> * Refactor: move null check earlier for better efficiency Co-authored-by: DedeHai <6280424+DedeHai@users.noreply.github.com> --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: DedeHai <6280424+DedeHai@users.noreply.github.com>
This commit is contained in:
@@ -1283,7 +1283,8 @@ function updateSelectedPalette(s)
|
||||
if (selElement) selElement.classList.remove('selected');
|
||||
|
||||
var selectedPalette = parent.querySelector(`.lstI[data-id="${s}"]`);
|
||||
if (selectedPalette) parent.querySelector(`.lstI[data-id="${s}"]`).classList.add('selected');
|
||||
if (!selectedPalette) return; // palette not yet loaded (custom palette on initial load)
|
||||
selectedPalette.classList.add('selected');
|
||||
|
||||
// Display selected palette name on button in simplified UI
|
||||
let selectedName = selectedPalette.querySelector(".lstIname").innerText;
|
||||
|
||||
Reference in New Issue
Block a user