search

Virtual-Royaume / Royaume-Frontend

Website of our community.
https://royaume.world
6 stars 0 forks source link

Ordre aléatoire des têtes du Royaume #59

Closed Steellgold closed 3 months ago

Steellgold commented 1 year ago

J'ai remarqué que les photos de profiles était tout le temps aligné dans le même ordre et je trouve sa pas très esthétique

J'ai donc fait en sorte qu'elles soit mélangé pour ajouter un peu plus de réalisme

Avant vs Après image image

Capture d'écrans prise au moment du rechargement de page

Steellgold commented 1 year ago

L'objectif de ce genre de données c'est juste d'avoir une preview du résultat final sans ajouter de complexité au code.

Si on en ajoute trop, le refactor sera + complexe quand on va connecter le site à l'API.

Je pense pas que cela rend + difficile au passage à l'API c'est simplement un tableau qu'on donne à la fonction donc il sera du même type qu'actuellement

Bluzzi commented 1 year ago

Ça peut porter à confusion pour le dev qui s’en occupera, il ne sera pas quoi faire de cette fonction.

Pareil si on crée ce genre de fonction, il faut l’accompagner de test unitaire. Aucune fonction de ce type ne doit pas en avoir.

Steellgold commented 1 year ago

Ça peut porter à confusion pour le dev qui s’en occupera, il ne sera pas quoi faire de cette fonction.

Pareil si on crée ce genre de fonction, il faut l’accompagner de test unitaire. Aucune fonction de ce type ne doit pas en avoir.

@Bluzzi Si je fait un test unitaire et que la fonction je la met donc à part dans un dossier (surement /utils) cela peut aider ?

Bluzzi commented 1 year ago

C'est déjà + ok, ça m'embête de prendre du temps sur un truc pas forcément permanent, mais oui tu peux.

Steellgold commented 1 year ago

C'est déjà + ok, ça m'embête de prendre du temps sur un truc pas forcément permanent, mais oui tu peux.

@Bluzzi Ce n'est pas pertinent mais pour un visiteur lambda se serra + beau que répéter dans le même ordre constant