aboutsummaryrefslogtreecommitdiff
path: root/scripts/theme-studio/browser-gates.js
diff options
context:
space:
mode:
authorCraig Jennings <c@cjennings.net>2026-06-15 17:30:03 -0500
committerCraig Jennings <c@cjennings.net>2026-06-15 17:30:03 -0500
commit7a7b1c164867c60325a8b9644a777e4e5cdf52ce (patch)
tree46abd912f8db28dcebb91296e00f62521226954c /scripts/theme-studio/browser-gates.js
parent5f915271f8d5da97ec86b44d6233872960092653 (diff)
downloaddotemacs-7a7b1c164867c60325a8b9644a777e4e5cdf52ce.tar.gz
dotemacs-7a7b1c164867c60325a8b9644a777e4e5cdf52ce.zip
refactor(theme-studio): drop the per-row reset column from package faces
Package faces was the only tier with a per-row reset button, in its own column. The syntax and ui tiers rely on the bulk reset alone, and the column cost horizontal space the table needs. I removed the per-row cell and its header, so the bulk reset (next to "lock all") is the single reset path now, matching the other tiers. A #viewtest assertion confirms the package rows carry no per-row reset button. The two weren't equivalent: the per-row button reset one face to its default, the bulk reset clears every unlocked face in the app. Losing single-face reset is the accepted tradeoff.
Diffstat (limited to 'scripts/theme-studio/browser-gates.js')
-rw-r--r--scripts/theme-studio/browser-gates.js1
1 files changed, 1 insertions, 0 deletions
diff --git a/scripts/theme-studio/browser-gates.js b/scripts/theme-studio/browser-gates.js
index 2b85fc8bb..c47251a59 100644
--- a/scripts/theme-studio/browser-gates.js
+++ b/scripts/theme-studio/browser-gates.js
@@ -671,6 +671,7 @@ if(location.hash==='#viewtest'){let ok=true;const notes=[];const A=(c,n)=>{if(!c
const firstApp=Object.keys(APPS)[0];sel.value=firstApp;onViewChange();
A(!vis('view-code')&&!vis('view-ui')&&vis('view-pkg'),'pkg-view-only');
A(curApp()===firstApp,'curApp-returns-selected-app');
+ A(!document.querySelector('#pkgbody .sbtn[title="reset to default"]'),'no-per-row-reset-button');
}
document.title='VIEWTEST '+(ok?'PASS':'FAIL');
const d=document.createElement('div');d.id='viewtest';d.textContent='VIEWTEST '+(ok?'PASS':'FAIL')+(notes.length?' fails='+notes.join(','):'');document.body.appendChild(d);}