LouanBen / wordle-fr

"Le Mot" un Wordle en français.
https://wordle.louan.me
GNU General Public License v3.0
127 stars 40 forks source link

Ajoute bépo aux dispositions de clavier #53

Closed ariasuni closed 2 years ago

ariasuni commented 2 years ago

Disposition: image

Choix de la disposition avec un grand écran: image

Choix de la disposition avec un petit écran: image

netlify[bot] commented 2 years ago

✔️ Deploy Preview for wordle-fr-louan ready!

🔨 Explore the source changes: 02000432652762fad05d37f9515f7293acf1667e

🔍 Inspect the deploy log: https://app.netlify.com/sites/wordle-fr-louan/deploys/62328838cf2a9700093f2d06

😎 Browse the preview: https://deploy-preview-53--wordle-fr-louan.netlify.app/

richie3366 commented 2 years ago

Merci beaucoup pour ta super contrib @ariasuni !

Quelques questions/suggestions :

En tout cas, à titre personnel je suis favorable à cette PR et à ce qu'elle soit mergée après concertation & mise en accord sur les points que je viens d'évoquer.

ariasuni commented 2 years ago

je ne suis pas bépoète, mais je suppose que tu l'es, et que tu pourras répondre en connaissance de cause : est-ce que le changement de place du W pose un souci de saisie ou c'est supportable car c'est une lettre rare ?

Personnellement, si les lettres ne sont plus alignées je suis perdue, alors je préfère largement le w déplacé à un endroit non-conventionnel.

je sais que les accents sont des caractères valides dans les noms de variables, classes & propriétés, mais je ne peux m'empêcher de penser qu'il serait préférable de ne pas en mettre (et d'en laisser que dans la valeur de name). Qu'en penses-tu ?

Je crois que la valeur de name est utilisée pour la classe CSS, donc je l’ai changé aussi.

richie3366 commented 2 years ago

Désolé pour le très gros délai de merge (que je vais faire peu après avoir posté ce commentaire). Je m'étais entretenu avec @Reelwens pour qu'il approuve la cohérence au niveau du design, et notamment de quelques changements de CSS. Il avait approuvé tout en faisant quelques remarques sur le sélecteur de claviers, en plus d'une question sur le bépo. J'ai répondu mais il n'a donné suite (il était probablement occupé et a dû oublier) et je n'ai pas pensé à le relancer.

Néanmoins puisqu'il était ok avec tes ajouts/changements, je considère que ta PR peut être merge sans devoir attendre davantage. :)

ariasuni commented 2 years ago

Si jamais il y a d’autres questions sur le bépo, je suis disposée à y répondre :)