Conversation
Size changesDetails📦 Next.js Bundle Analysis for react-devThis analysis was generated by the Next.js Bundle Analysis action. 🤖 This PR introduced no changes to the JavaScript bundle! 🙌 |
tdd
left a comment
There was a problem hiding this comment.
Salut @nina-alin !
Avant toute chose, merci pour tout ce boulot ! C'est super cool de continuer à nous filer un coup de main.
L'essentiel des tweaks vient de "portal" -> "portail" (à toi de propager les tweaks manquants, je ne les ai faits que si je touchais à autre chose) et à "en-dehors" (y'a un tiret).
Quelques glissements sémantiques et éclaircissements, mais sinon super 👍
Avant de traiter les retours, lis bien TOUT ce message, notamment les consignes d'intégration optimale des retours par lot(s)
Tout est fait sous forme de Suggestions GitHub, il te suffit à chaque suggestion que tu approuves de
cliquer le bouton d'ajout au lot de modifs à committer (pas de commit direct !), et sur la fin de créer le commit groupé. (ce mode de regroupement en un seul commit est à favoriser absolument, mais tu devras aller dans l'onglet Files changed pour y accéder, il n'est pas utilisable dans l’onglet Conversation)
Si certaines appellent discussion, n'hésite pas, tu réponds, on en parle, on avance ensemble 🤝
Quand tout sera bouclé côté code, je ferai une repasse visuelle finale en local avant de fusionner. J'y recalerai si besoin les usual suspects : espaces insécables avant les ponctuations doubles, et apostrophes françaises.
Encore mille mercis, j'ai hâte de voir cette PR finalisée et dans main !
|
Hello @tdd ! J'ai bien commit tes suggestions et j'ai replacé |
|
Coucou @nina-alin ! OK super je fais une dernière repasse visuelle, un rebase sur le Merci à toi ! 🙏 |
of "rendu"
Co-authored-by: Christophe Porteneuve <tdd@tddsworld.com>
ebe5c49 to
df93073
Compare
Un des composants de
react-domque j'adore utiliser 🛩️