fix(meteogram): retours de mise en prod (échelle T°, pression, icônes, dates) - #117
Open
cmer81 wants to merge 2 commits into
Open
fix(meteogram): retours de mise en prod (échelle T°, pression, icônes, dates)#117cmer81 wants to merge 2 commits into
cmer81 wants to merge 2 commits into
Conversation
…bande fixe Retours de mise en prod : - windbarbs sur un axe Y dédié invisible : leurs valeurs (m/s, seuil 0) entraient dans les extrêmes de l'axe des températures et l'écrasaient dès que la série vent était visible - axe T° : startOnTick/endOnTick désactivés (l'arrondi au tick entier donnait 0-50° pour une courbe 25-32°) + tickPixelInterval 40 (le défaut 72px pouvait ne laisser qu'une graduation dans le tiroir) - pression : pleine opacité et 1,5px au lieu de rgba 0.55 / 1px (« terne ») - icônes météo : bande fixe en haut façon yr.no au lieu de suivre la courbe de T° (bizarre courbe masquée, empiétait sur les histos de précip) ; sélection pure weatherSymbolPlacements testée Vérifié en headless (playwright + swiftshader) : échelle identique vent visible/masqué, bande d'icônes stable T° masquée. Claude-Session: https://claude.ai/code/session_01Mz1tPVV9HepH5ndmMXXTWm
« mer. 15 juil. » débordait de son créneau de tick → Highcharts passait tous les labels de l'axe X en biais, avec le premier tronqué (« mer. 15 juil.me… »). Le format court « mer. 15 » tient horizontal ; le mois complet reste lisible dans l'encart de valeurs et l'en-tête du tiroir. Vérifié en headless : 0 label pivoté, 0 tronqué sur 19 (horizon 5 j). Claude-Session: https://claude.ai/code/session_01Mz1tPVV9HepH5ndmMXXTWm
Deploying with
|
| Status | Name | Latest Commit | Preview URL | Updated (UTC) |
|---|---|---|---|---|
| ✅ Deployment successful! View logs |
infoclimat-numeric | f55727c | Commit Preview URL Branch Preview URL |
Jul 15 2026, 03:40 PM |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Contexte
Retours du premier testeur après la mise en prod du météogramme (PR #114) — les 4 points sont traités.
Changements
1. Échelle des T° écrasée dès que les barbules sont visibles
La série
windbarbn'avait pas d'axe Y déclaré → elle partageait l'axe des températures, et sesvalue(m/s, seuil 0) entraient dans le calcul des extrêmes. Fix : axe Y dédié invisible (les barbes sont dessinées àyOffsetfixe, l'axe ne sert à rien visuellement).En vérifiant, un second coupable est apparu : même sans barbules, les défauts Highcharts
startOnTick/endOnTickarrondissent les extrêmes au tick entier — une courbe 25–32 °C donnait un axe 0–50°. Désactivés, +tickPixelInterval: 40(le défaut 72px pouvait ne laisser qu'une seule graduation dans le tiroir).2. Courbe de pression « terne »
Pleine opacité
#fbbf24et 1,5px (au lieu dergba(…, 0.55)/ 1px). Le pointillé est conservé comme signature (distinction avec la T°).3. Icônes de nébulosité collées à la courbe de T°
Bande fixe en haut de la zone de tracé, façon yr.no — elles suivaient la courbe (héritage de la démo Highcharts) : bizarre quand la courbe est masquée via la légende, et par T° basse elles descendaient sur les histogrammes de précip et leurs valeurs. La sélection des pas (stride adaptatif, codes null écartés) est extraite en fonction pure
weatherSymbolPlacements(testée) ; le rendu utilisetoPixels+ demi-pasbetween(découplé de la série T°).4. Affichage des dates
« mer. 15 juil. » débordait de son créneau de tick → Highcharts passait tous les labels de l'axe X en biais, premier label tronqué (« mer. 15 juil.me… »). Date courte « mer. 15 » (toujours en gras) → axe 100 % horizontal. Le mois complet reste lisible dans l'encart de valeurs et l'en-tête du tiroir.
Vérification
meteogram-chart.test.ts,weather-symbols.test.ts) — 472 tests ✔, svelte-check ✔, lint ✔.claude/rules/components.mdmises à jour (5 axes Y, bande fixe)https://claude.ai/code/session_01Mz1tPVV9HepH5ndmMXXTWm