pict-section-fileview 1.0.0 → 1.0.1

package/LICENSE

@@ -0,0 +1,21 @@
+MIT License
+
+Copyright (c) 2026 Steven Velozo
+
+Permission is hereby granted, free of charge, to any person obtaining a copy
+of this software and associated documentation files (the "Software"), to deal
+in the Software without restriction, including without limitation the rights
+to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
+copies of the Software, and to permit persons to whom the Software is
+furnished to do so, subject to the following conditions:
+
+The above copyright notice and this permission notice shall be included in all
+copies or substantial portions of the Software.
+
+THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
+AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
+OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
+SOFTWARE.

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "pict-section-fileview",
-    "version": "1.0.0",
+    "version": "1.0.1",
     "description": "A universal, embeddable file viewer: one control that detects a file's type (from a Sluice record or a bare range-served URL) and dispatches it to the right renderer — code, markdown, images (ordinary + enormous), PDF, video and audio — delegating each media kind to a purpose-built child section.",
     "main": "source/Pict-Section-Fileview.js",
     "scripts": {

package/source/providers/Pict-Fileview-Provider.js

@@ -470,6 +470,22 @@ class PictFileviewProvider extends libPictProvider
 		this.pict.ContentAssignment.assignContent(this.toolbarSelector, tmpHTML);
 	}
 
+	/**
+	 * True when the stage element exists AND still holds mounted content. The reuse-one-per-kind fast-paths in
+	 * _mountResolved() assume the previously mounted DOM is still on the page -- but a host that re-stamps our mount
+	 * point on its own re-render (e.g. the fileview host div lives inside the consumer view's template, so every
+	 * consumer render() wipes it) leaves _currentHandle pointing at detached nodes. There "same descriptor -> return"
+	 * and the in-place update() would both paint nothing (blank stage). When the stage was wiped this returns false so
+	 * _mountResolved falls through to a clean destroy + re-mount. DOM-less contexts (SSR / unit tests) report true so
+	 * they never force a churn the environment cannot observe.
+	 */
+	_isStagePopulated()
+	{
+		if (typeof document === 'undefined') { return true; }
+		let tmpStageElement = document.querySelector(this.stageSelector);
+		return !!(tmpStageElement && tmpStageElement.children && tmpStageElement.children.length > 0);
+	}
+
 	_mountResolved()
 	{
 		const tmpStage = this.stageSelector;
@@ -499,11 +515,15 @@ class PictFileviewProvider extends libPictProvider
 		}
 
 		// Reuse-one-per-kind. Pict has no removeView, so re-minting a child per file leaks views/observers. When
-		// the resolved renderer is the one already mounted:
+		// the resolved renderer is the one already mounted AND its mounted DOM is still on the page:
 		//   - same descriptor identity (a bare re-render) → nothing to do;
 		//   - new file, renderer exposes update() → re-feed the live child in place, no teardown;
 		//   - otherwise → fall through to a clean destroy + re-mount.
-		if (this._currentHandle && this._currentHandle.Hash === tmpRenderer.Hash)
+		// The _isStagePopulated() guard is essential: a consumer that re-stamps our host on its own re-render wipes the
+		// stage out from under us, detaching the mounted child. Without the guard both fast-paths leave a blank stage;
+		// with it we fall through to a re-mount and repaint. (A consumer with a stable host never trips it, so normal
+		// file-to-file switching still reuses the child via update() with no churn.)
+		if (this._currentHandle && this._currentHandle.Hash === tmpRenderer.Hash && this._isStagePopulated())
 		{
 			if (this._currentHandle.Descriptor === this._descriptor) { return; }
 			if (typeof this._currentHandle.update === 'function')