Skip to content

fix: align the assemble ricochet styles with the other ricochets - #82

Merged
Lunik merged 1 commit into
developfrom
fix/ricochet-assembler-etats
Oct 16, 2026
Merged

Lunik merged 1 commit into
developfrom
fix/ricochet-assembler-etats

Conversation

@Lunik

@Lunik Lunik commented Oct 15, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Selection of a brick in the "Assembler" ricochet now uses the shared .prise state (Papier 2 background, thin Encre inset) instead of the orange ring.
  • Verdict reuses the shared states: correct brick in Encre with a checked circle, wrong chosen brick hatched/dashed with a struck circle, others dimmed.
  • Assembly slots take the same juste / rate states; the .rico-brique-ko class is removed.

Checks

  • npm run typecheck, npm test (194 passed), check_identity on ricochets.css: 0 errors.
  • Not yet checked visually in a browser.

🤖 Generated with Claude Code

https://claude.ai/code/session_01T8T5x563byynGcaX5z95uV


Generated by Claude Code

…e other ricochets

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01T8T5x563byynGcaX5z95uV

Lunik commented Oct 15, 2026

Copy link
Copy Markdown
Contributor Author

CI rouge : API et tâches (Rust) → étape « Audit the dependencies » (cargo audit)

Ce n’est pas ce changement : cette PR ne touche que web/ (TSX et CSS). Les étapes format, clippy, tests, .sqlx et contrat OpenAPI passent, ainsi que PWA (TypeScript). L’audit échoue sur 4 avis RustSec contre rustls 0.23.45 (RUSTSEC-2026-0346, -0347, -0348, -0349), tous corrigés en >= 0.23.46. Les derniers runs de develop sont rouges aussi.

Aucun correctif n’existe encore dans le dépôt. Un patch minimal a été testé en local : cargo update -p rustls --precise 0.23.46 ne change que 3 lignes de Cargo.lock. Je ne l’ajoute pas ici pour ne pas élargir cette PR ; il vaut mieux le passer dans sa propre PR vers develop.


Generated by Claude Code

@Lunik
Lunik marked this pull request as ready for review October 16, 2026 11:08
@Lunik
Lunik merged commit a9b5b05 into develop Oct 16, 2026
2 of 3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants