Corrige une régression fonctionnelle : revient à Bulma 1.0.2 (plus de Sass)
L'utilisateur a signalé des bugs réels en jeu après la refonte design system : jauges qui ne se remplissent plus, conditions de visibilité cassées sur une carte (enveloppe ouverte/fermée affichées en même temps). Racine du problème : Phase A avait basculé Bulma de 1.0.2 (CDN d'origine) vers 0.9.4, seule version compatible avec libsass (1.0.2 utilise le système de modules @use/@forward que libsass ne sait pas compiler). Or screens/rendering/render_jauge.py pilote le remplissage d'une jauge en fixant en ligne --bulma-progress-value-background-color — une variable CSS qui n'existe QUE dans le nouveau système de theming de Bulma 1.x, absente de 0.9.4. Aucune perte de données : les champs d'objet étaient toujours intacts en base (vérifié directement sur projects/test/game.db) — uniquement un problème de rendu/comportement en jeu. Correction : revient à Bulma 1.0.2, vendoré tel quel et non modifié (static/vendor/bulma.min.css, ~677 Ko, auto-hébergé — toujours aucune dépendance CDN). Bulma 1.x expose déjà tout son thème via de vraies variables CSS (--bulma-primary-h/-s/-l, --bulma-radius...), justement conçues pour être surchargées après coup SANS recompilation Sass — styles/bulma-override.css les redéfinit avec la palette Forge (teintes HSL calculées à partir des couleurs de la charte). build_css.py devient un simple concaténage de 4 fichiers (bulma.min.css + bulma-override.css + forge-tokens.css + forge-custom.css), plus besoin de libsass ni d'aucun compilateur — supprimé de requirements.txt. styles/bulma/ (source Sass 0.9.4 vendorée en Phase A) et styles/forge-theme.scss supprimés. Nouvelle règle ajoutée en commentaire dans bulma-override.css : ne plus jamais changer de version de Bulma sans `grep -rn "\-\-bulma-" screens/ templates/` d'abord — cette dépendance n'est pas que visuelle. 197 tests toujours verts (ils ne couvrent que le HTML généré, jamais le rendu réel — c'est pour ça que cette régression n'avait pas été détectée avant que l'utilisateur ne la signale en jouant pour de vrai). Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
This commit is contained in:
co-authored by
Claude Sonnet 5
parent
eaebc555d3
commit
c60884ec46
+29
-14
@@ -1,26 +1,41 @@
|
||||
"""Compile styles/forge-theme.scss (tokens Forge + Bulma vendoré en local,
|
||||
voir regles/FORGE_ENGINE_TEMPLATE_BULMA.md) en static/style.css — le seul
|
||||
fichier CSS chargé par les templates (un seul <link>, inchangé). Pas de
|
||||
build automatique : lancer `python build_css.py` après toute modification
|
||||
sous styles/."""
|
||||
"""Assemble static/style.css à partir de styles/ — un simple concaténage,
|
||||
PAS une compilation Sass (voir styles/bulma-override.css pour l'incident
|
||||
qui a motivé ce choix : Bulma 1.x expose déjà tout son thème via des
|
||||
variables CSS standard, aucun compilateur n'est nécessaire pour le
|
||||
recolorer). Ordre important — chaque fichier ne fait que redéfinir des
|
||||
variables/règles posées par le précédent :
|
||||
1. static/vendor/bulma.min.css — Bulma 1.0.2 vendoré tel quel, non
|
||||
modifié (voir son en-tête pour la provenance).
|
||||
2. styles/bulma-override.css — variables --bulma-* de la charte Forge.
|
||||
3. styles/forge-tokens.css — tokens --forge-*/--accent... de l'app.
|
||||
4. styles/forge-custom.css — CSS custom de l'app (floatPanel, canvas,
|
||||
éditeurs...), consomme les tokens ci-dessus.
|
||||
|
||||
Lancer `python build_css.py` après toute modification sous styles/ ou
|
||||
static/vendor/bulma.min.css."""
|
||||
import os
|
||||
|
||||
import sass
|
||||
|
||||
_BASE_DIR = os.path.dirname(os.path.abspath(__file__))
|
||||
_ENTRY = os.path.join(_BASE_DIR, "styles", "forge-theme.scss")
|
||||
_STYLES_DIR = os.path.join(_BASE_DIR, "styles")
|
||||
_OUTPUT = os.path.join(_BASE_DIR, "static", "style.css")
|
||||
|
||||
_SOURCES = [
|
||||
os.path.join(_BASE_DIR, "static", "vendor", "bulma.min.css"),
|
||||
os.path.join(_STYLES_DIR, "bulma-override.css"),
|
||||
os.path.join(_STYLES_DIR, "forge-tokens.css"),
|
||||
os.path.join(_STYLES_DIR, "forge-custom.css"),
|
||||
]
|
||||
|
||||
|
||||
def build():
|
||||
css = sass.compile(
|
||||
filename=_ENTRY,
|
||||
include_paths=[os.path.join(_BASE_DIR, "styles")],
|
||||
output_style="expanded",
|
||||
)
|
||||
chunks = []
|
||||
for path in _SOURCES:
|
||||
with open(path, encoding="utf-8") as f:
|
||||
chunks.append(f"/* ---- {os.path.relpath(path, _BASE_DIR)} ---- */\n" + f.read())
|
||||
css = "\n\n".join(chunks)
|
||||
with open(_OUTPUT, "w", encoding="utf-8") as f:
|
||||
f.write(css)
|
||||
print(f"Compilé : {_ENTRY} -> {_OUTPUT} ({len(css)} octets)")
|
||||
print(f"Assemblé : {len(_SOURCES)} fichiers -> {_OUTPUT} ({len(css)} octets)")
|
||||
|
||||
|
||||
if __name__ == "__main__":
|
||||
|
||||
Reference in New Issue
Block a user