From b148d2455c1a90b96e1c4df524169029b53bd2d7 Mon Sep 17 00:00:00 2001 From: "@gapmiss" Date: Sat, 30 May 2026 16:39:53 -0500 Subject: [PATCH] fix: use document instead of activeDocument for hover class Obsidian's new settings window is a separate document, so activeDocument was targeting the wrong body when toggling the reveal-on-hover setting. Co-Authored-By: Claude Opus 4.5 --- main.ts | 11 +++++++++-- 1 file changed, 9 insertions(+), 2 deletions(-) diff --git a/main.ts b/main.ts index a2e9db4..4f64086 100644 --- a/main.ts +++ b/main.ts @@ -130,7 +130,8 @@ export default class BlurPlugin extends Plugin { } onunload(): void { - activeDocument.body.removeClass('obsidian-blur-hover'); + // eslint-disable-next-line obsidianmd/prefer-active-doc -- must target main window + document.body.removeClass('obsidian-blur-hover'); } async loadSettings(): Promise { @@ -142,7 +143,13 @@ export default class BlurPlugin extends Plugin { } applyHoverClass(): void { - activeDocument.body.toggleClass('obsidian-blur-hover', this.settings.revealOnHover); + // eslint-disable-next-line obsidianmd/prefer-active-doc -- must target main window, not settings window + const body = document.body; + if (this.settings.revealOnHover) { + body.addClass('obsidian-blur-hover'); + } else { + body.removeClass('obsidian-blur-hover'); + } } blurBlockHandler(source: string, el: HTMLElement, _ctx: MarkdownPostProcessorContext): void {