Move theme workspace into dedicated Appearance tab

Author: FolderView Plus Test

archive/folderview.plus-2026.04.12.01.txz.sha256

@@ -0,0 +1 @@
+75b34c1de9f07fac20cc10e67364b2598f1d1e04302bbcea664ae549538b5428  folderview.plus-2026.04.15.03.txz

archive/folderview.plus-2026.04.12.01.txz archive/folderview.plus-2026.04.15.03.txzarchive/folderview.plus-2026.04.12.01.txz renamed to archive/folderview.plus-2026.04.15.03.txz

@@ -6,13 +6,17 @@
 <!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.04.15.02">
-<!ENTITY md5 "1ceddf3e9c607e9b80deefa768bafabb">
+<!ENTITY version "2026.04.15.03">
+<!ENTITY md5 "bc8b75aff9dcb40180ac80bc5eb10874">
 ]>
 
 <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.04.15.03
+- UX: Settings workspace layout, section flows, and table behavior.
+
+
 ###2026.04.15.02
 - Fix: Fixed concurrent JSON writes during first-load settings normalization so VM and Docker prefs no longer race on the same temporary file.
 - Fix: Updated atomic JSON writes to use a unique temp file per request before rename, which prevents intermittent `Failed to replace JSON payload` errors during settings bootstrap and diagnostics reads.

archive/folderview.plus-2026.04.15.03.txz.sha256

@@ -79,10 +79,10 @@ if (!empty($fvplusRuntimeConflicts)) {
 <span id="update-check-status" class="status-line"></span>
 <span id="rollback-status" class="status-line"></span>
 
+<h2 data-fv-section="theme-workspace" data-fv-advanced="1" data-fv-advanced-group="appearance">Theme workspace</h2>
 <div id="fv-theme-workspace-panel" class="fv-theme-workspace-panel">
     <div class="rules-panel fv-theme-workspace-shell">
         <div class="rules-header">
-            <h3>Theme Workspace</h3>
             <span class="rules-help">Import GitHub CSS themes, activate one managed theme at a time, and layer FolderView Plus token overrides and custom CSS on top.</span>
         </div>
         <div class="fv-theme-workspace-body">

folderview.plus.plg

@@ -16,6 +16,7 @@ const LEGACY_ADVANCED_SECTION_KEYS = [
     'conflict-inspector'
 ];
 const ADVANCED_SECTION_KEYS = new Set([
+    'theme-workspace',
     'auto-assignment',
     'bulk-assignment',
     'runtime-actions',
@@ -25,18 +26,20 @@ const ADVANCED_SECTION_KEYS = new Set([
     'diagnostics',
     'conflict-inspector'
 ]);
-const ADVANCED_GROUPS = ['automation', 'rules', 'recovery', 'operations', 'diagnostics'];
+const ADVANCED_GROUPS = ['automation', 'rules', 'recovery', 'operations', 'appearance', 'diagnostics'];
 const ADVANCED_GROUP_LABELS = {
     automation: 'Automation',
     rules: 'Rules',
     recovery: 'Recovery',
     operations: 'Operations',
+    appearance: 'Appearance',
     diagnostics: 'Diagnostics'
 };
 const SECTION_APPLY_BEHAVIOR = Object.freeze({
     customizations: 'instant',
     docker: 'instant',
     vms: 'instant',
+    'theme-workspace': 'instant',
     'auto-assignment': 'instant',
     'conflict-inspector': 'instant',
     'bulk-assignment': 'instant',
@@ -47,6 +50,7 @@ const SECTION_APPLY_BEHAVIOR = Object.freeze({
     diagnostics: 'instant'
 });
 const ADVANCED_GROUP_BY_SECTION = {
+    'theme-workspace': 'appearance',
     'auto-assignment': 'rules',
     'bulk-assignment': 'automation',
     'conflict-inspector': 'rules',
@@ -69,6 +73,7 @@ const ADVANCED_MODULE_KEYS_BY_TAB = Object.freeze({
     rules: Object.freeze([]),
     recovery: Object.freeze(['docker_backups', 'vm_backups', 'change_history']),
     operations: Object.freeze(['docker_templates', 'vm_templates']),
+    appearance: Object.freeze([]),
     diagnostics: Object.freeze(['change_history'])
 });
 const BASIC_WORKSPACE_SECTION_KEYS = new Set(['docker', 'vms']);

src/folderview.plus/usr/local/emhttp/plugins/folderview.plus/FolderViewPlus.page

@@ -96,6 +96,7 @@ test('settings page loads extracted settings metadata before the main runtime',
 });
 
 test('settings page exposes theme workspace and saved folder defaults controls', () => {
+    assert.match(page, /<h2 data-fv-section="theme-workspace" data-fv-advanced="1" data-fv-advanced-group="appearance">Theme workspace<\/h2>/);
     assert.match(page, /id="fv-theme-workspace-panel"/);
     assert.match(page, /id="fv-theme-github-source"/);
     assert.match(page, /id="fv-theme-workspace-list"/);

src/folderview.plus/usr/local/emhttp/plugins/folderview.plus/scripts/folderviewplus.settings-sections.js

@@ -206,7 +206,7 @@ test('advanced settings split auto-assignment rules into a dedicated Rules tab',
     assert.match(settingsPage, /<h2 data-fv-section="auto-assignment" data-fv-advanced="1" data-fv-advanced-group="rules">Auto-assignment rules<\/h2>/);
     assert.match(settingsPage, /<h2 data-fv-section="conflict-inspector" data-fv-advanced="1" data-fv-advanced-group="rules">Rule testing and troubleshooting<\/h2>/);
     assert.match(settingsPage, /<h2 data-fv-section="bulk-assignment" data-fv-advanced="1" data-fv-advanced-group="automation">Bulk assignment<\/h2>/);
-    assert.match(settingsSectionsJs, /const ADVANCED_GROUPS = \['automation', 'rules', 'recovery', 'operations', 'diagnostics'\];/);
+    assert.match(settingsSectionsJs, /const ADVANCED_GROUPS = \['automation', 'rules', 'recovery', 'operations', 'appearance', 'diagnostics'\];/);
     assert.match(settingsSectionsJs, /rules:\s*'Rules'/);
     assert.match(settingsSectionsJs, /'auto-assignment':\s*'rules'/);
     assert.match(settingsSectionsJs, /'conflict-inspector':\s*'rules'/);
@@ -219,6 +219,13 @@ test('advanced settings split auto-assignment rules into a dedicated Rules tab',
     assert.ok(bulkAssignmentIndex > conflictInspectorIndex, 'bulk assignment should remain after the Rules sections');
 });
 
+test('theme workspace lives in its own Appearance advanced tab', () => {
+    assert.match(settingsPage, /<h2 data-fv-section="theme-workspace" data-fv-advanced="1" data-fv-advanced-group="appearance">Theme workspace<\/h2>/);
+    assert.match(settingsSectionsJs, /appearance:\s*'Appearance'/);
+    assert.match(settingsSectionsJs, /'theme-workspace':\s*'appearance'/);
+    assert.match(settingsSectionsJs, /appearance:\s*Object\.freeze\(\[\]\)/);
+});
+
 test('rules tab uses a source-switched workspace and bulk assignment keeps the two-column desktop layout', () => {
     assert.match(settingsPage, /class="fv-rules-source-switch"[\s\S]*setRulesWorkspaceType\('docker'\)[\s\S]*setRulesWorkspaceType\('vm'\)/);
     assert.match(settingsPage, /class="rules-panel fv-rules-workspace" data-fv-rules-type="docker"[\s\S]*id="docker-rules-status"[\s\S]*id="docker-rules-selection-summary"[\s\S]*id="docker-rules"/);