refactor: make sure to follow OO24 guidelines for performance

- only register event listener when view is ready
- check for instance of view when accessing it

manifest.json

@@ -1,7 +1,7 @@
 {
 	"id": "journal-review",
 	"name": "Journal Review",
-	"version": "2.4.2",
+	"version": "2.4.3",
 	"minAppVersion": "0.15.0",
 	"description": "Review your daily notes on their anniversaries, like \"what happened today last year\".",
 	"author": "Kageetai",

package.json

@@ -1,6 +1,6 @@
 {
 	"name": "obsidian-plugin-journal-review",
-	"version": "2.4.2",
+	"version": "2.4.3",
 	"description": "Review your daily notes on their anniversaries, like \"what happened today last year\"\n\n",
 	"main": "main.js",
 	"scripts": {

src/main.ts

@@ -96,7 +96,9 @@ export default class JournalReviewPlugin extends Plugin {
 		}
 
 		// "Reveal" the leaf in case it is in a collapsed sidebar
-		workspace.revealLeaf(leaf!);
+		await workspace.revealLeaf(leaf!);
+
+		this.renderView();
 	}
 
 	onunload() {
@@ -127,14 +129,17 @@ export default class JournalReviewPlugin extends Plugin {
 		this.settings = Object.assign({}, DEFAULT_SETTINGS, parsedData);
 	}
 
-	async saveSettings() {
-		await this.saveData(this.settings);
+	renderView() {
+		const leaf = this.app.workspace.getLeavesOfType(VIEW_TYPE).first();
 
-		// rerender view
-		(
-			this.app.workspace.getLeavesOfType(VIEW_TYPE)[0]?.view as OnThisDayView
-		)?.renderView();
+		if (leaf && leaf.view instanceof OnThisDayView) {
+			leaf.view.renderView();
+		}
+	}
 
+	async saveSettings() {
+		await this.saveData(this.settings);
+		this.renderView();
 		this.setupFocusListener();
 	}
 }

src/view.tsx

@@ -20,49 +20,51 @@ export default class OnThisDayView extends ItemView {
 
 		this.settings = settings;
 
-		this.registerEvent(
-			this.app.vault.on("create", (file: TFile) => {
-				if (getDateFromFile(file, "day")) {
-					this.debouncedRenderView();
-				}
-			}),
-		);
-		this.registerEvent(
-			this.app.vault.on("delete", (file: TFile) => {
-				if (getDateFromFile(file, "day")) {
-					this.debouncedRenderView();
-				}
-			}),
-		);
-		this.registerEvent(
-			this.app.vault.on("rename", (file: TFile) => {
-				if (getDateFromFile(file, "day")) {
-					this.debouncedRenderView();
-				}
-			}),
-		);
-
-		this.registerEvent(
-			this.app.workspace.on("file-open", (file) => {
-				const dateFromFile = file && getDateFromFile(file, "day");
+		this.app.workspace.onLayoutReady(() => {
+			this.registerEvent(
+				this.app.vault.on("create", (file: TFile) => {
+					if (getDateFromFile(file, "day")) {
+						this.debouncedRenderView();
+					}
+				}),
+			);
+			this.registerEvent(
+				this.app.vault.on("delete", (file: TFile) => {
+					if (getDateFromFile(file, "day")) {
+						this.debouncedRenderView();
+					}
+				}),
+			);
+			this.registerEvent(
+				this.app.vault.on("rename", (file: TFile) => {
+					if (getDateFromFile(file, "day")) {
+						this.debouncedRenderView();
+					}
+				}),
+			);
 
-				if (this.settings.renderOnFileSwitch && dateFromFile) {
-					this.debouncedRenderView(dateFromFile);
-				}
-			}),
-		);
+			this.registerEvent(
+				this.app.workspace.on("file-open", (file) => {
+					const dateFromFile = file && getDateFromFile(file, "day");
 
-		// rerender at midnight
-		this.registerInterval(
-			window.setInterval(
-				() => {
-					if (new Date().getHours() === 0) {
-						this.renderView();
+					if (this.settings.renderOnFileSwitch && dateFromFile) {
+						this.debouncedRenderView(dateFromFile);
 					}
-				},
-				60 * 60 * 1000,
-			),
-		);
+				}),
+			);
+
+			// rerender at midnight
+			this.registerInterval(
+				window.setInterval(
+					() => {
+						if (new Date().getHours() === 0) {
+							this.renderView();
+						}
+					},
+					60 * 60 * 1000,
+				),
+			);
+		});
 	}
 
 	getViewType() {

versions.json

@@ -20,5 +20,6 @@
 	"2.3.2": "0.15.0",
 	"2.4.0": "0.15.0",
 	"2.4.1": "0.15.0",
-	"2.4.2": "0.15.0"
+	"2.4.2": "0.15.0",
+	"2.4.3": "0.15.0"
 }