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

Wordle visual redesign #30

Closed Reelwens closed 2 years ago

Reelwens commented 2 years ago

Proposition de nouveau design pour WordleFR 🎨

Capture d'écran des maquettes d'interface de l'évolution proposée :

image

netlify[bot] commented 2 years ago

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

🔨 Explore the source changes: b03a3156e2618c14c0dbc5c11dd94407d7045484

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

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

richie3366 commented 2 years ago

C'est super bien fait, j'adore !

Je suggère (à voir si @LouanBen est du même avis) d'ajouter un remerciement à ton nom pour tout ce travail de redesign. Cela pourrait se faire en changeant celui en place :

Merci à Richie pour son aide précieuse !

par

Merci à Richie pour son aide précieuse, ainsi qu'à Reelwens pour le nouveau design !

(à toi de choisir vers quoi tu veux linker ton nom)

Sinon, il n'y a qu'une seule chose qui me perturbe, c'est la taille de "LE MOT" en header, je trouve qu'il est un peu petit. Je me demande comment ça rendrait en plus gros.

Merci en tout cas pour avoir passé de ton temps sur cette super PR. :)

Reelwens commented 2 years ago

Hello

Merci @richie3366 pour ton commentaire, ça me fait très plaisir 😊

J'ai fait des modifications supplémentaires suites à quelques tests (merci @PLhery) :

LouanBen commented 2 years ago

Coucou @Reelwens !

J'adore, ça rend super bien, bravo !

Je vais être tatillon sur quelques détails de responsive à modifier si possible :

Merci encore à toi, contacte moi si t'as le moindre soucis ou si t'as besoin de précisions !

Reelwens commented 2 years ago

Merci pour tes retours @LouanBen, à vrai dire ne ne savais pas qu'il existait des devices si petits !

Je me suis occupé des problèmes relevés :

J'en ai aussi profité pour augmenter légèrement la hauteur de la taille des touches de clavier sur les tailles d'écran qui le permettent, j'ai remarqué après tests que ça offrait une bien meilleure UX :)

@Richie3366 & @iamsabbirsobhani : I integrated the PR-31 in my own, to simplify the merge and PR process, thank you for your contribution 😊