feat(division): intro enrichie — partage visuel en lots + compte#49
Open
isc wants to merge 2 commits into
Open
feat(division): intro enrichie — partage visuel en lots + compte#49isc wants to merge 2 commits into
isc wants to merge 2 commits into
Conversation
L'intro division réutilisait la grille de points telle quelle (juste a×b animé). On la rend pédagogique, conforme aux références (modèle array / equal-groups relié à la table, cf. Van de Walle, specs §11.4) : une fois la grille remplie, les rangées se séparent en LOTS distincts (le diviseur), puis un lot est mis en avant et son compte (= le quotient) est dévoilé — c'est la réponse, par partage équitable. - DotGrid : nouveau prop opt-in `groupReveal`. Après remplissage : étape `grouped` (rangées → pastilles/lots) puis `counted` (un lot en avant + son compte qui pop). Aucun impact sur les usages multiplication (prop absent). - SessionScreen (intro div) : passe `groupReveal` + légende « On partage N en D lots égaux — combien dans chaque ? » (la grille répond, l'astuce facteur- manquant relie à la table juste en dessous). - Guide : capture 16-division-intro attend la révélation du compte (état final). Vérifié visuellement (12 ÷ 6 → 6 lots de 2). 161 tests, tsc + lint OK. https://claude.ai/code/session_011fdoBoa4wtauybCcDVPuGt
- Délais de la révélation "lots" nommés (GROUP_DELAY_MS / COUNT_DELAY_MS) au lieu de 350/1300 en dur dans l'effet. - Condition de highlight allégée : `grouped && counted` → `counted` (counted survient toujours après grouped via les timers, donc `grouped &&` redondant). Écartés : hook useSequenceTimeout (2 effets simples), keyframe/pilule honey partagées (hors diff, params distincts), attribut data- pour le guide (il cible déjà des classes internes), orchestration côté SessionScreen / i18n (sur- ingénierie). Aucun changement de comportement. 161 tests, tsc + lint OK. https://claude.ai/code/session_011fdoBoa4wtauybCcDVPuGt
Contributor
|
Preview déployée pour cette PR :
Commit : |
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
2 participants
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.
L'intro d'un nouveau fait de division réutilisait la grille de points telle quelle (juste
a × banimé, comme la multiplication). On la rend pédagogique et conforme aux références (modèle array / equal-groups relié à la table connue — Van de Walle, Make Math Moments, PBS LearningMedia ; specs §11.4).Animation (modèle partitif « partage »)
Une fois la grille remplie :
Choix tranchés : séparation en rangées-pastilles (plutôt que distribuer point par point, trop long pour ≤81 points) ; compte qui pop d'un coup sur le lot mis en avant.
Implémentation
DotGrid: nouveau prop opt-ingroupReveal. Après remplissage : phasegrouped(rangées → lots) puiscounted(lot en avant + compte qui pop), via deux délais nommés. Mirroir du précédentshowRotation. Zéro impact sur les usages multiplication (intro, commutativité, feedback, image mystère) — prop absent → aucun timer/classe/DOM en plus.groupReveal+ la légende (classe existante.session-intro-explanation).16-division-introattend la révélation du compte (état final enrichi).Vérification
Vérifié visuellement (capture du guide,
12 ÷ 6 → 6 lots de 2). ✅tsc -b· ✅ lint · ✅ 161 tests (+2 : rendu du badge de compte, et absence sansgroupReveal)./simplifyappliqué (délais nommés, condition allégée).https://claude.ai/code/session_011fdoBoa4wtauybCcDVPuGt
Generated by Claude Code