@massimo-cassandro/minimo 1.38.3 → 1.38.4

package/dev-tools/design-tokens-builder/build-tokens-src/light-dark.mjs

@@ -4,7 +4,9 @@
 // and a `dark` mode, custom properties present in BOTH — matched by name —
 // are written once, as `--name: light-dark(<light-value>, <dark-value>);`,
 // instead of being split across the base mode's top-level `:root { ... }`
-// block and a `@media (prefers-color-scheme: dark) { ... }` block.
+// block and a `@media (prefers-color-scheme: dark) { ... }` block. If the two
+// values are identical, light-dark() would be redundant: the bare value is
+// written instead, e.g. `--name: <value>;`.
 //
 // Only `light` and `dark` are affected:
 //   - a third mode (e.g. a custom `high-contrast` key in sourceModes) is left
@@ -74,7 +76,12 @@ export const applyLightDarkFunc = ({ modeFinalProps, modeNames, baseMode, select
     // Prefers the light-side trailing comment (e.g. a token $description);
     // falls back to the dark-side one if light has none.
     const comment = light.comment || dark.comment;
-    sharedProps[name] = `light-dark(${light.value}, ${dark.value});${comment ? ` ${comment}` : ''}`;
+    // Equal light/dark values: light-dark(x, x) would be redundant — use the
+    // bare value directly instead.
+    const value = light.value === dark.value
+      ? light.value
+      : `light-dark(${light.value}, ${dark.value})`;
+    sharedProps[name] = `${value};${comment ? ` ${comment}` : ''}`;
   }
 
   const stripCommon = (props) => {

package/dev-tools/starter-kit/source_files/root/webpack.config.mjs

@@ -138,7 +138,7 @@ const { CustomPropsPurgeCssPlugin } = runCustomPropsPlugin
 // NB: PurgeCSS elabora ogni asset css singolarmente, quindi i @keyframes
 // definiti nello `shared.css` ma usati solo in un altro asset (es. un css di
 // progetto che usa una animation di minimo) verrebbero rimossi: in tal caso
-// aggiungerli a `safelist.keyframes` o impostare `keyframes: false` in
+// aggiungerli a 'safelist.keyframes' o impostare `keyframes: false` in
 // `purgeCSSOptions`
 const shared_chunk_paths = (module) => {
 
@@ -146,7 +146,6 @@ const shared_chunk_paths = (module) => {
   const sep = '[\\\\/]'; // stringa che produce [\\/] nel pattern
   const pathsRegexp = new RegExp([
     'node_modules',
-    'app/src',
     'app/src/js',
     'app/src/web-components',
     'app/src/components',
@@ -411,8 +410,7 @@ const config = {
     //   fileName: path.join(output_dir, 'manifest.json'),
     // sf: fileName: 'manifest.json', // scrive in output.path
     //   // basePath: item.source_dirname
-    //   // removeKeyHash: /(^(_assets\/(?!(fonts\/))))|((\?as_asset)$)/,
-    //   removeKeyHash: true, // /([a-f0-9]{32}\.?)/gi, // /(\?as_asset)$/,
+    //   // removeKeyHash: /\?.*$/, // /([a-f0-9]{32}\.?)/gi, // /(\?as_asset)$/,
     //   // rimuove i font dal manifest. Non necessari, rendono il file inutilmente grande
     //   filter: isDevelopment? undefined : (FileDescriptor) => {
     //     return /fonts/.test(FileDescriptor.path)? false : true;

package/dev-tools/starter-kit/webpack-config-upg.md

@@ -8,6 +8,10 @@ Le versioni precedenti sono conservate in `archived/webpack-config-v2/` del repo
 
 ---
 
+## Fix da verificare sempre
+* WebpackManifestPlugin: impostare `removeKeyHash: /\?.*$/, // /([a-f0-9]{32}\.?)/gi, // /(\?as_asset)$/,`, sostituendo eventuali impostazione `removeKeyHash: true` se presente
+
+
 ## Da v2 a v3
 
 Novità della v3: le definizioni delle custom properties non sono più gestite da PurgeCSS (`variables`) né da postcss-jit-props, ma da un plugin webpack dedicato, `custom-props-purgecss-plugin.mjs`, che opera sugli asset CSS finali (dopo la minificazione) e inietta solo le custom properties effettivamente usate (dipendenze transitive incluse), partendo dal file master `app/css/custom-properties.css`. Il plugin è attivabile con un flag, indipendente da `usePurgeCss`.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@massimo-cassandro/minimo",
-  "version": "1.38.3",
+  "version": "1.38.4",
   "description": "Light css framework & utilities",
   "type": "module",
   "main": "./index.js",