Merge fix/macro-icons: Fix macro icons not showing up (PR #149)

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
This commit is contained in:
John Morabito 2026-06-13 11:37:47 -04:00
commit c8207d8693

View file

@ -144,16 +144,17 @@ export function updateMacroCommands(plugin: CommanderPlugin): void {
);
for (const command of oldCommands) {
//@ts-ignore
app.commands.removeCommand(command);
plugin.app.commands.removeCommand(command);
}
const macros = plugin.settings.macros;
for (const [idx, macro] of Object.entries(macros)) {
for (const [idx, macro] of macros.entries()) {
plugin.addCommand({
id: `macro-${idx}`,
name: macro.name,
icon: macro.icon,
callback: () => {
plugin.executeMacro(parseInt(idx));
plugin.executeMacro(Number(idx));
},
});
}
@ -195,7 +196,7 @@ export function injectIcons(
if (command) {
command.icon = mapped.iconID;
} else {
settings.mappedIcons.remove(mapped);
settings.mappedIcons = settings.mappedIcons.filter(m => m !== mapped);
}
});
}