fix: encodage accents docs section 5 + conversion auto des facteurs au changement de design#109
Merged
Conversation
…u changement de design
Deux corrections de bugs visibles côté utilisatrice :
1. docs.jsx section 5 — encodage des accents cassé
Le commentaire ajouté dans la PR précédente avait ses 'é' encodés
comme 'é' littéraux dans le JSX (bug d'encodage à la sauvegarde),
apparaissant à l'écran comme « utilis|u00e9s », « modalit|u00e9s »,
etc. Corrigé en utilisant la convention {'é'} JSX cohérente avec le
reste du fichier généré.
2. DesignConfigurator — changer le type de design ne convertit pas
les facteurs existants
Bug rapporté : « quand je passe de inter à intra, ça ne change rien
et je ne peux même pas ajouter/sélectionner mes contrebalancements ».
Mécanisme : les facteurs créés sous le type d'origine (ex. BETWEEN)
conservaient leur type, et le sélecteur de type de facteur n'apparaît
qu'en design MIXED. hasWithinFactors restait donc false et la section
Contrebalancement restait cachée.
Fix : quand on bascule vers un design PUR (BETWEEN ou WITHIN), tous
les facteurs existants sont automatiquement convertis vers ce type.
Un toast informe l'utilisatrice du nombre de facteurs convertis.
Pour MIXED, les facteurs sont laissés tels quels.
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
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
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.
Summary
Deux corrections de bugs visibles côté utilisatrice :
1. docs.jsx section 5 — accents cassés (cf. screenshot)
Le commentaire que j'avais ajouté à la PR #108 avait ses caractères `é` rendus comme des chaînes ASCII `\u00e9` littérales dans le JSX (bug d'encodage à la sauvegarde côté Edit tool). À l'écran, ça s'affichait « utilis|u00e9s », « modalit|u00e9s », « ajout|u00e9es », etc.
Corrigé en utilisant la convention `{'é'}` JSX, cohérente avec le reste du fichier généré.
2. DesignConfigurator — changer le type de design ne convertit pas les facteurs existants
Bug rapporté : « quand je passe de inter à intra, ça ne change rien et je ne peux même pas ajouter/sélectionner mes contrebalancements ».
Mécanisme :
Fix : quand on bascule vers un design PUR (`BETWEEN` ou `WITHIN`), tous les facteurs existants sont automatiquement convertis vers ce type. Un `toast` informe l'utilisatrice du nombre de facteurs convertis.
Pour `MIXED`, les facteurs sont laissés tels quels (assignation manuelle par facteur via le sélecteur qui ne s'affiche qu'en MIXED — c'est cohérent).
Test plan