Trim docker tooltip trigger fix for perf budget

Author: alexphillips-dev

archive/folderview.plus-2026.03.29.17.txz.sha256

@@ -0,0 +1 @@
+fa5a00ce5d8d98d73628b1d6dcec1624e24546f6ac52740931532ec93736f221  folderview.plus-2026.03.30.17.txz

archive/folderview.plus-2026.03.29.17.txz archive/folderview.plus-2026.03.30.17.txzarchive/folderview.plus-2026.03.29.17.txz renamed to archive/folderview.plus-2026.03.30.17.txz

@@ -6,13 +6,19 @@
 <!ENTITY launch "Settings/FolderViewPlus">
 <!ENTITY plugdir "/usr/local/emhttp/plugins/&name;">
 <!ENTITY pluginURL "https://raw.githubusercontent.com/&github;/dev/folderview.plus.plg">
-<!ENTITY version "2026.03.30.16">
-<!ENTITY md5 "f6601bdadd11448fdc6a1e62c302d0a2">
+<!ENTITY version "2026.03.30.17">
+<!ENTITY md5 "b08a5b0aa4659eef6a3e4eafd57de99c">
 ]>
 
 <PLUGIN name="&name;" author="&author;" version="&version;" launch="&launch;" pluginURL="&pluginURL;" icon="folder-icon.png" support="https://forums.unraid.net/topic/197631-plugin-folderview-plus/" min="7.0.0">
     <CHANGES>
 
+###2026.03.30.17
+- Fix: Stop Docker advanced preview context popups configured for click from eager-opening on the first hover interaction.
+- Quality: Trim the lazy tooltip trigger-mode fix so the Docker runtime stays inside the repo perf budget while preserving hover-mode eager-open behavior.
+- Regression guard: Cover lazy tooltip initialization so click-trigger folders only eager-open on click or touch, while hover-trigger folders still open on hover.
+
+
 ###2026.03.30.16
 - Fix: Stop Docker advanced preview context popups configured for click from eager-opening on the first hover interaction.
 - Regression guard: Cover lazy tooltip initialization so click-trigger folders only eager-open on click or touch, while hover-trigger folders still open on hover.

archive/folderview.plus-2026.03.30.17.txz.sha256

@@ -1692,7 +1692,7 @@ const buildDockerTooltipContent = (ct) => {
     return $content;
 };
 
-const initializeDockerTooltipOnDemand = ($target, init, options = {}) => {
+const initializeDockerTooltipOnDemand = ($target, init, hoverOpen = true) => {
     if (DOCKER_PREVIEW_POPUP_ENABLED !== true) {
         return;
     }
@@ -1703,18 +1703,13 @@ const initializeDockerTooltipOnDemand = ($target, init, options = {}) => {
         return;
     }
     $target.data('fvTooltipsterDeferred', true);
-    const eagerOpenEventTypes = new Set(
-        (Array.isArray(options?.openOnEventTypes) ? options.openOnEventTypes : ['mouseenter', 'click', 'touchstart'])
-            .map((eventType) => String(eventType || '').trim().toLowerCase())
-            .filter(Boolean)
-    );
     const ensureInitialized = (eventType = '') => {
         if ($target.data('fvTooltipsterInitialized') === true) {
             return;
         }
         $target.data('fvTooltipsterInitialized', true);
         init();
-        if (eagerOpenEventTypes.has(eventType)) {
+        if (eventType !== 'mouseenter' || hoverOpen) {
             setTimeout(() => {
                 try {
                     $target.tooltipster('open');
@@ -1725,7 +1720,7 @@ const initializeDockerTooltipOnDemand = ($target, init, options = {}) => {
         }
     };
     $target.one('mouseenter.fvLazyTooltip click.fvLazyTooltip touchstart.fvLazyTooltip', (event) => {
-        ensureInitialized(String(event?.type || '').trim().toLowerCase());
+        ensureInitialized(event?.type || '');
     });
 };
 // Advanced preview popups are opt-in per folder; keep the runtime lazy so
@@ -3957,11 +3952,7 @@ const createFolder = (folder, id, positionInMainOrder, liveOrderArray, container
                         }
                     },
                     content: $('<div class="fv-tooltip-lazy-loading">Loading preview...</div>')
-                }), {
-                    openOnEventTypes: triggerMode === 'hover'
-                        ? ['mouseenter', 'click', 'touchstart']
-                        : ['click', 'touchstart']
-                });
+                }), triggerMode === 'hover');
             } else if (FOLDER_VIEW_DEBUG_MODE && tooltip_trigger_element && tooltip_trigger_element.length > 0) {
                 console.log(`[FV3_DEBUG] createFolder (id: ${id}), container ${ct.shortId}: FolderView preview popup runtime is disabled; skipping tooltip initialization.`);
             } else {

folderview.plus.plg

@@ -31,13 +31,13 @@ const extractArrowFunctionBody = (source, signature) => {
 
 const initializeDockerTooltipOnDemandBody = extractArrowFunctionBody(
     dockerJs,
-    'const initializeDockerTooltipOnDemand = ($target, init, options = {}) => '
+    'const initializeDockerTooltipOnDemand = ($target, init, hoverOpen = true) => '
 );
 
 const initializeDockerTooltipOnDemand = new Function(
     '$target',
     'init',
-    'options',
+    'hoverOpen',
     'DOCKER_PREVIEW_POPUP_ENABLED',
     'setTimeout',
     initializeDockerTooltipOnDemandBody
@@ -100,7 +100,7 @@ test('docker lazy tooltip init does not eager-open click-trigger tooltips on fir
         () => {
             initCount += 1;
         },
-        { openOnEventTypes: ['click', 'touchstart'] },
+        false,
         true,
         runImmediateTimeout
     );
@@ -121,7 +121,7 @@ test('docker lazy tooltip init still eager-opens hover-trigger tooltips on first
         () => {
             initCount += 1;
         },
-        { openOnEventTypes: ['mouseenter', 'click', 'touchstart'] },
+        true,
         true,
         runImmediateTimeout
     );
@@ -141,7 +141,7 @@ test('docker lazy tooltip init eager-opens click-trigger tooltips on first click
         () => {
             initCount += 1;
         },
-        { openOnEventTypes: ['click', 'touchstart'] },
+        false,
         true,
         runImmediateTimeout
     );

src/folderview.plus/usr/local/emhttp/plugins/folderview.plus/scripts/docker.js

@@ -228,8 +228,9 @@ test('docker preview popup runtime stays enabled behind lazy advanced-preview in
     assert.match(dockerJs, /const DOCKER_PREVIEW_POPUP_ENABLED = true;/);
     assert.match(dockerJs, /fvTooltipLazyBuilt/);
     assert.match(dockerJs, /Loading preview\.\.\./);
-    assert.match(dockerJs, /const initializeDockerTooltipOnDemand = \(\$target,\s*init,\s*options = \{\}\) =>/);
-    assert.match(dockerJs, /openOnEventTypes: triggerMode === 'hover'/);
+    assert.match(dockerJs, /const initializeDockerTooltipOnDemand = \(\$target,\s*init,\s*hoverOpen = true\) =>/);
+    assert.match(dockerJs, /initializeDockerTooltipOnDemand\(\$\(tooltip_trigger_element\), \(\) => \$\(tooltip_trigger_element\)\.tooltipster\(\{/);
+    assert.match(dockerJs, /\}\), triggerMode === 'hover'\);/);
     assert.match(dockerJs, /if \(DOCKER_PREVIEW_POPUP_ENABLED !== true\) \{\s*return;\s*\}/);
     assert.match(dockerJs, /if\(DOCKER_PREVIEW_POPUP_ENABLED && tooltip_trigger_element && tooltip_trigger_element\.length > 0\) \{/);
 });