obsidian-plugin/binxly_Atomizer
1
0
Fork 0
mirror of https://github.com/binxly/Atomizer.git synced 2026-07-22 05:31:35 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

Fix notice management and improve status bar updates

Browse source 
Move loading notice creation inside confirmation callback so it's only
shown after user confirms the action. Previously the notice reference
was created before confirmation, which could cause issues.

Improve status bar to reflect actual plugin state:
- Set to "Processing..." when work starts
- Set to "Ready" on success
- Set to "Error" on failure
- Remove status update from finally block to allow proper error state

This ensures users get accurate feedback about what the plugin is doing
and eliminates the bug where status always showed "Ready" even after errors.

Fixes H2 (Notice management bug) and H3 (Status bar always "Ready")
from CODE_REVIEW.md

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
This commit is contained in:
Mark Ayers 2025-10-12 19:47:23 -04:00
parent 635b1c6421
commit 2e16a3c11e
1 changed files with 3 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
src/main.ts
View file

@ -75,6 +75,7 @@ export default class AtomizerPlugin extends Plugin {
}
new ConfirmationModal(this.app, async () => {
this.statusBarItem.setText("Atomizer: Processing...");
const loadingNotice = new Notice("Processing note...", 0);
try {
@ -119,11 +120,12 @@ export default class AtomizerPlugin extends Plugin {
new Notice(
`Successfully created ${atomicNotes.length} atomic notes`,
);
this.statusBarItem.setText("Atomizer: Ready");
} catch (error) {
this.handleError(error);
this.statusBarItem.setText("Atomizer: Error");
} finally {
loadingNotice.hide();
this.statusBarItem.setText("Atomizer: Ready");
}
}).open();
}