@ryuu-64/dsh-find-all 0.1.8 → 0.1.9

package/CHANGELOG.md

@@ -1,5 +1,12 @@
 # Changelog
 
+## 0.1.9
+
+- Reveal host-owned `hidden="until-found"` content before scrolling to a search match, instead of treating its layout geometry as proof that the match is visible.
+- Wait for concealed matches to become visible before marking navigation as successful.
+
+The supported host-version list is unchanged.
+
 ## 0.1.8
 
 - Keep Ctrl+F and the header search button bound to the current main conversation across sidebar focus changes and main-view navigation.

package/lib/client.js

@@ -727,6 +727,16 @@ window.__ModuleLoader__.load({
 			return null;
 		}
 
+		function hiddenUntilFoundAncestor(node, root) {
+			var element = node && (node.nodeType === 1 ? node : node.parentElement);
+			for (; element && root && root.contains(element); element = element.parentElement) {
+				var hidden = element.getAttribute && element.getAttribute('hidden');
+				if (typeof hidden === 'string' && hidden.toLowerCase() === 'until-found') return element;
+				if (element === root) break;
+			}
+			return null;
+		}
+
 		function captureReadingOrigin() {
 			var target = state.target, viewport = readingViewport(target);
 			var store = state.adapter && state.adapter.nodes(target);
@@ -850,13 +860,18 @@ window.__ModuleLoader__.load({
 				if (token !== state.navigationToken || !originValid(origin)) return;
 				var viewport = readingViewport(origin.target);
 				var current = state.ranges[state.index];
-				var geometry = current && origin.target.root.contains(current.startContainer) && rangeRect(current);
-				if (current && !geometry) revealMatchRange(origin.target, current, activated);
+				var attached = current && origin.target.root.contains(current.startContainer);
+				// hidden=until-found uses content-visibility:hidden, so Chromium may
+				// report layout geometry for descendants that are not painted. Reveal
+				// the host-owned disclosure before trusting Range geometry.
+				var concealed = attached && hiddenUntilFoundAncestor(current.startContainer, origin.target.root);
+				var geometry = attached && !concealed && rangeRect(current);
+				if (current && (!geometry || concealed)) revealMatchRange(origin.target, current, activated);
 				if (geometry && scrolledRange !== current) {
 					scrollToRange(current);
 					scrolledRange = current;
 				}
-				if (viewport && current && origin.target.root.contains(current.startContainer) && rangeRect(current, viewport)) {
+				if (viewport && attached && !concealed && rangeRect(current, viewport)) {
 					var firstNavigation = !origin.navigated;
 					origin.navigated = true;
 					updateReturnButton();

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ryuu-64/dsh-find-all",
-  "version": "0.1.8",
+  "version": "0.1.9",
   "description": "Ctrl+F find bar for the DSH desktop app: it searches the whole conversation, not just the messages already on screen.",
   "type": "module",
   "license": "MIT",