diff --git a/luci-theme-footstrap/Makefile b/luci-theme-footstrap/Makefile index 896a3775..10242b03 100644 --- a/luci-theme-footstrap/Makefile +++ b/luci-theme-footstrap/Makefile @@ -17,7 +17,7 @@ LUCI_NAME:=luci-theme-footstrap FOOTSTRAP_VERSION?= ifneq ($(FOOTSTRAP_VERSION),) PKG_VERSION:=$(FOOTSTRAP_VERSION) -PKG_RELEASE:=4 +PKG_RELEASE:=5 endif LUCI_TITLE:=Footstrap Theme diff --git a/luci-theme-footstrap/htdocs/luci-static/resources/fs-prefs.js b/luci-theme-footstrap/htdocs/luci-static/resources/fs-prefs.js index e71db5ef..044aa478 100644 --- a/luci-theme-footstrap/htdocs/luci-static/resources/fs-prefs.js +++ b/luci-theme-footstrap/htdocs/luci-static/resources/fs-prefs.js @@ -303,8 +303,10 @@ function colorAxis(key, attr, hueProp, colorProp) { * EXPLICITLY — including the default, so it overrides a router default (see the header; lsDel would * mean "inherit") — and remove the property AT the default, so 02-tokens' own value shows through. * They differ ONLY in how the number formats onto the property (px vs a 0..2 multiplier), so that is - * the one argument that varies. The sd() field name is passed in explicitly: unlike enum/hue it is - * NOT the key minus 'fs-' ('fs-radius' -> rounding, 'fs-tint-strength' -> tint_strength). */ + * the one argument that varies. The sd() field name is passed in explicitly because ONE instance + * needs a RENAME rather than a spelling: 'fs-radius' -> rounding. The other four would fall out of + * the same hyphen fold enumAxis and colorAxis do ('fs-tint-strength' -> tint_strength), but a + * factory that is right for four keys out of five is the trap those two walked into. */ function propAxis(key, sdKey, prop, min, max, dfl, fmt) { const inRange = (n) => (typeof n === 'number' && n >= min && n <= max); const def = () => { const d = sd(sdKey); return inRange(d) ? d : dfl; }; diff --git a/luci-theme-footstrap/htdocs/luci-static/resources/fs-sheets.js b/luci-theme-footstrap/htdocs/luci-static/resources/fs-sheets.js index 556a71b9..17dd5b6a 100644 --- a/luci-theme-footstrap/htdocs/luci-static/resources/fs-sheets.js +++ b/luci-theme-footstrap/htdocs/luci-static/resources/fs-sheets.js @@ -518,12 +518,18 @@ function silence(el) { * * A foreign sheet is injected by ONE page and has no business painting any other. Before this, an * invasive sheet made the whole document spent (documentPoisoned) and the SPA fell back to a full - * load on the way OUT — correct, but stock LuCI pays it: `view/status/cpu.js` and its four realtime - * siblings each `document.head.append(E('style', …))` at MODULE EVAL, and that style carries - * `svg text { fill: #eee; font-size: 9pt }` — real properties on a bare selector, so invasive by the - * only definition that catches `[class] { padding: 0 !important }`. Measured on the router: 37 of 38 - * pages navigated in-place instead of 38, `status/realtime/load` 15 ms -> 157 ms, and the whole - * realtime family 2-10x slower. + * load on the way OUT — correct, and paid by ORDINARY pages: `luci-app-filemanager`, a stock app, + * calls its `insertCss()` at MODULE EVAL and lands two