@klodd/ds 3.3.0 → 3.4.0

package/references/05-open-decisions/0006-bar-kind-template-migration-CLOSED.md

@@ -0,0 +1,75 @@
+# 0006 - Ekonom bar_kind()-helper - migration till kategori-id for colored-bar
+
+## Status
+**Closed 2026-05-08** via Option A (utoka bar_kind() till kategori-id).
+Ekonom-commit [4ea9415](https://github.com/drawn124578/ekonom/commit/4ea9415).
+
+bar_kind() i routers/kategorier.py returnerar nu cat.key (mat, el,
+vatten, etc.) istallet for variant-modifiers ('default'/'wealth'/'accent').
+ekonom.css .cat-bar-fill far ny background: var(--bar-accent, var(--text-tertiary))
+sa --bar-accent (satt av .cat-bar-{kid}-modifier i ekonom-domain.css)
+faktiskt blir bakgrund. 8 distinkta kategori-farger istallet for 3
+generic varianter. Pytest 636/636.
+
+## Context
+v3.1.0 introducerade paketets `.colored-bar`-komponent + Ekonoms
+bindings-klasser `.cat-bar-mat` / `.cat-bar-el` ... / `.cat-bar-privat`
+som mappar app-tokens (--cat-X-icon) till paketets `--bar-accent`.
+
+CSS-bindningarna ar redo att anvandas. Templates anvander dock idag
+ett ANNAT monster:
+
+```html
+<div class="cat-bar-track">
+  <div class="cat-bar-fill cat-bar-{{ bar_kind(g.category) }}"
+       data-bar-width="{{ pct }}%"></div>
+</div>
+```
+
+Python-helpern `bar_kind(category)` returnerar idag en av tre
+**variant-modifiers**: `default`, `wealth`, `accent`. Inte kategori-id.
+
+Aktivering av det nya kategori-monstret kraver att antingen:
+- `bar_kind()` utokas att returnera kategori-id (mat, el, noje osv.)
+- Templates byter klassnamn fran `cat-bar-{kind}` till
+  `colored-bar cat-bar-{category-id}`
+
+## Considered options
+
+### Option A: Utoka bar_kind() att returnera kategori-id
+**Pros:**
+- Aktiverar det nya kategori-monstret fullt ut
+- Templates fortsatter anvanda `cat-bar-{{ bar_kind(g) }}`-syntax -
+  bara inneliggande varde andras
+- Visuellt rikare resultat (8 distinkta kategori-farger istallet for
+  3 generic varianter)
+**Cons:**
+- Andrar Python-logik pa `bar_kind()` - kraver test-uppdatering
+- Templates behover ocksa lagga till `colored-bar`-klassen parallellt:
+  `class="colored-bar cat-bar-{{ bar_kind(g) }}"`
+- Behover bestamma vad bar_kind() returnerar for icke-kategori-rader
+  (wealth, accent-varianter for special-data)
+
+### Option B: Behall variant-modifiers som de ar
+**Pros:**
+- Inga template- eller Python-andringar
+- Default/wealth/accent-monstret ar etablerat och fungerar
+- CSS-bindningarna lever passivt for framtida anvandning utan att
+  blockera nuvarande funktionalitet
+**Cons:**
+- Det nya kategori-monstret ar redo i CSS men oanvant
+- Inga visuella forbattringar pa kategori-bars
+
+## Pending decision criteria
+- Sprint-prioritet: ar visuellt rikare kategori-fargning vard att kosta
+  Python-logik-andring + template-edit?
+- Samordning med MADR 0001 (.tx-row -> .list-row): bagge migrationer
+  beror Ekonoms transaction-listor och kan delas pa samma sprint for
+  effektivitet
+
+## Notes
+- CSS-bindningarna `.cat-bar-X` lever sedan v3.1.0 (Ekonom-commit
+  f011c2f). Inga paket-andringar behovs - bara app-side adoption
+- Fortsatt fungerande default/wealth/accent-monster betyder INGEN
+  forsamring vid val av Option B
+- Ingen tidsgrans pa beslutet

package/references/05-open-decisions/0007-ekonom-base-import-gap-CLOSED.md

@@ -0,0 +1,111 @@
+# 0007 - Ekonoms main.css-kedja importerar inte paketets ds/base/-mapp
+
+## Status
+**Closed 2026-05-07** via Option A med selektiv import. Ekonoms
+main.css importerar nu paketets `ds/base/layout.css` direkt. Skip
+av `pwa.css` (dubblett mot Ekonoms egen) och `typography.css`
+(Inter Google Fonts-konflikt med Sprint D2).
+
+## Context
+Jubb och Ekonom har olika CSS-arkitektur for hur paketets `base/`-mapp
+laddas:
+
+**Jubb:** paketets `base/`-filer kopieras till `app/static/css/base/`
+(rotnivå). `main.css` importerar `base/typography.css` direkt:
+
+```css
+/* Jubbs main.css */
+@import url("ds/base.css");
+@import url("base/typography.css");   /* paketets fil, rotniva */
+@import url("ds/components.css");
+...
+```
+
+**Ekonom:** paketets `base/`-filer kopieras till `app/static/css/ds/base/`-
+undermapp (for att inte krocka med Ekonoms egna `app/static/css/base/`
+filer). `main.css` importerar via `ds/bundles/ekonom.css` som bara
+inkluderar Ekonom-specifika `ds/base.css`, INTE paketets nyare filer:
+
+```css
+/* Ekonoms main.css */
+@import url("ds/bundles/ekonom.css");
+
+/* ds/bundles/ekonom.css */
+@import url("../ekonom-domain.css");
+@import url("../base.css");          /* Ekonoms egen, inte paketet */
+@import url("../components.css");
+@import url("../utilities.css");
+@import url("../ekonom.css");
+@import url("../components/inline-edit.css");
+```
+
+`ds/base/typography.css`, `ds/base/pwa.css`, `ds/base/layout.css`
+(paketets versioner) kopieras till disk via build:ds men laddas aldrig.
+
+### Konsekvens
+
+Framtida paket-uppdateringar i `base/`-mappen kraver manuella hot-fixes
+i Ekonom. Konkret exempel: v3.2.0 introducerade Inter @import i
+`base/typography.css`. Jubb fick det automatiskt - Ekonom kravde commit
+`5d59f1c` med Inter @import direkt i `main.css`.
+
+## Considered options
+
+### Option A: Lagg till explicita @import-rader i Ekonoms main.css
+```css
+/* Ekonoms main.css */
+@import url("ds/base/typography.css");
+@import url("ds/base/layout.css");
+@import url("ds/bundles/ekonom.css");
+```
+
+**Pros:**
+- Enklast
+- Minst risk
+- Paketets base/-filer aktiveras automatiskt vid build:ds
+- Bibehaller Ekonoms befintliga ds/-undermapp-arkitektur
+**Cons:**
+- Lite duplicering om paketet utvidgas med ytterligare base/-filer
+  (man maste komma ihag att lagga till @import)
+- `base/pwa.css` skips eftersom den redan laddas via base.html-link
+
+### Option B: Skriv om build:ds-scriptet att kopiera + auto-importera
+build:ds genererar en `ds/base/_index.css` som listar alla `base/-`
+filer i paketet. main.css importerar `ds/base/_index.css`.
+
+**Pros:**
+- Helt automatisk - inga manuella @import-rader behovs
+**Cons:**
+- Mer komplex build-script
+- Genererad fil i ds/-mappen som ingen redigerar manuellt - cognitive
+  overhead
+
+### Option C: Harmonisera Ekonoms mappstruktur med Jubbs
+Flytta paketets `base/`-filer till Ekonoms `app/static/css/base/`-
+rotniva (overskriver Ekonoms egna). Kraver migration av Ekonoms
+befintliga `base/pwa.css` och `base/typography.css`.
+
+**Pros:**
+- Identisk arkitektur Jubb/Ekonom
+- Inga manuella @import behovs
+**Cons:**
+- Mer invasivt - kraver hantering av krockande filnamn
+- Ekonoms befintliga `app/static/css/base/typography.css` har Ekonom-
+  specifika styling-regler som inte ar i paketet
+- Storre risk for visuell regression
+
+## Recommendation
+**Option A.** Enklast, minst risk, kan ske i samma sprint som annan
+Ekonom-cleanup (eventuellt MADR 0001 .tx-row -> .list-row eller MADR
+0002 parallel-classnames-cleanup).
+
+## Pending decision criteria
+- Ar paketets `base/`-utokningar tillrackligt vanliga for att
+  rattfardiga Option B (auto-importer)?
+- Ar Ekonoms befintliga `base/`-mapp viktig att behalla, eller kan
+  Option C utforas i en cleanup-sprint?
+
+## Notes
+- Hot-fix via Option A ar redan delvis utford for Inter (commit 5d59f1c
+  i Ekonom lagger till Google Fonts @import direkt i main.css). Inte
+  ren Option A men en proxy-losning som fungerar