Trim palette names and remove redundant save button

This commit is contained in:
Jared Miller 2026-02-07 20:54:23 -05:00
parent e23ac18cd9
commit b46b1709b4
Signed by: shmup
GPG key ID: 22B5C6D66A38B06C

View file

@ -230,7 +230,6 @@
<select id="palette-select" style="width: 100%; margin-bottom: 0.25rem;"></select> <select id="palette-select" style="width: 100%; margin-bottom: 0.25rem;"></select>
<div style="display: flex; gap: 0.25rem;"> <div style="display: flex; gap: 0.25rem;">
<button id="palette-new" style="flex: 1; font-size: 0.85em; padding: 2px 4px;">new</button> <button id="palette-new" style="flex: 1; font-size: 0.85em; padding: 2px 4px;">new</button>
<button id="palette-save" style="flex: 1; font-size: 0.85em; padding: 2px 4px;">save</button>
<button id="palette-rename" style="flex: 1; font-size: 0.85em; padding: 2px 4px;">rename</button> <button id="palette-rename" style="flex: 1; font-size: 0.85em; padding: 2px 4px;">rename</button>
<button id="palette-delete" style="flex: 1; font-size: 0.85em; padding: 2px 4px;">delete</button> <button id="palette-delete" style="flex: 1; font-size: 0.85em; padding: 2px 4px;">delete</button>
</div> </div>
@ -663,34 +662,37 @@ ${paletteStr}"""
function createNewPalette() { function createNewPalette() {
const name = prompt('Enter palette name:'); const name = prompt('Enter palette name:');
if (!name) return; if (!name || !name.trim()) return;
const trimmedName = name.trim();
const palettes = loadPalettesFromStorage(); const palettes = loadPalettesFromStorage();
if (palettes[name]) { if (palettes[trimmedName]) {
alert('Palette with that name already exists.'); alert('Palette with that name already exists.');
return; return;
} }
palettes[name] = structuredClone(DEFAULT_PALETTE); palettes[trimmedName] = structuredClone(DEFAULT_PALETTE);
savePalettesToStorage(palettes); savePalettesToStorage(palettes);
loadPalette(name); loadPalette(trimmedName);
} }
function renamePalette() { function renamePalette() {
const newName = prompt('Enter new name:', activePaletteName); const newName = prompt('Enter new name:', activePaletteName);
if (!newName || newName === activePaletteName) return; if (!newName || !newName.trim()) return;
const trimmedName = newName.trim();
if (trimmedName === activePaletteName) return;
const palettes = loadPalettesFromStorage(); const palettes = loadPalettesFromStorage();
if (palettes[newName]) { if (palettes[trimmedName]) {
alert('Palette with that name already exists.'); alert('Palette with that name already exists.');
return; return;
} }
palettes[newName] = palettes[activePaletteName]; palettes[trimmedName] = palettes[activePaletteName];
delete palettes[activePaletteName]; delete palettes[activePaletteName];
savePalettesToStorage(palettes); savePalettesToStorage(palettes);
activePaletteName = newName; activePaletteName = trimmedName;
saveActivePaletteName(newName); saveActivePaletteName(trimmedName);
updatePaletteSelect(); updatePaletteSelect();
} }
@ -961,7 +963,6 @@ ${paletteStr}"""
}); });
document.getElementById('palette-new').addEventListener('click', createNewPalette); document.getElementById('palette-new').addEventListener('click', createNewPalette);
document.getElementById('palette-save').addEventListener('click', saveCurrentPalette);
document.getElementById('palette-rename').addEventListener('click', renamePalette); document.getElementById('palette-rename').addEventListener('click', renamePalette);
document.getElementById('palette-delete').addEventListener('click', deletePalette); document.getElementById('palette-delete').addEventListener('click', deletePalette);