Louis-htmlcss commented 1 week ago

🚀 Nouvelle Pull Request

Proposez vos modifications pour améliorer Papillon

Informations importantes

Merci de vous référer à la documentation sur la contribution si vous avez des questions à propos des pull requests (https://gitbook.getpapillon.xyz/organisation/outils-internes/github)

Checklist d'avant pull request

Veuillez cocher toutes les cases applicables en remplaçant [ ] par [x].

[x] Vous avez testé de build le projet avec vos modifications et ce build a réussi
[x] Vous respectez les conventions de codage et de nommage du projet
[x] Vous utilisez la tabulation pour l'indentation afin de maintenir un code lisible
[x] Cette pull request n'est pas un duplicata d'une autre
[x] Cette pull request est prête à être revue (review) et fusionnée (merge)
[x] Il n'y a pas de TODO (aka des annotations pour du code manquant) dans vos modifications
[x] Il n'y a pas d'erreurs de langue dans votre code (grammaire, vocabulaire, conjugaison, orthographe)
[x] Les détails des changements ont été décrits ci-dessous
[x] Cette pull-request n'est pas une "breaking-change" (des modifications qui vont entraîner la modification du fonctionnement de certaines fonctionnalités déjà existantes)

Changelogs proposés

Auparavant, le cadre entourant l'emoji de la matière était ovale plutôt que parfaitement circulaire. Cette modification ajuste le style du conteneur de l'emoji pour garantir une forme circulaire cohérente, améliorant ainsi l'esthétique et l'uniformité de l'affichage.

Changements principaux :

Remplacement du padding par des dimensions fixes (width et height)
Ajout de justifyContent et alignItems pour centrer l'emoji
Maintien du borderRadius à 50 pour conserver la forme circulaire

Rexxt commented 1 week ago

quel désastre

rien de testé, aucun screenshot

Rexxt commented 1 week ago

rien de coché dans la checklist même

cleeemeeent commented 1 week ago

t'as mm pas coché le truc d'avant pull request

cleeemeeent commented 1 week ago

quel désastre

rien de testé, aucun screenshot

Ça m'étonne mm plus

Rexxt commented 1 week ago

j'avoue j'avoue

Louis-htmlcss commented 1 week ago

C'est aléatoire aussi, ça me le fait 1 fois sur 2

cleeemeeent commented 1 week ago

de? Le rapport

Orafilynie commented 1 week ago

C'est aléatoire aussi, ça me le fait 1 fois sur 2

j'avoue tes PRs sont de plus en plus aléatoires la mon reuf

Louis-htmlcss commented 1 week ago

C'est aléatoire aussi, ça me le fait 1 fois sur 2

j'avoue tes PRs sont de plus en plus aléatoires la mon reuf

Je parle que ça soit ovale

tryon-dev commented 6 days ago

est ce que c'est possible d'avoir un screenshot de ce que tu as changé avant et apres stp

Rexxt commented 6 days ago

ça sert à rien la PR est mal complétée et elle comporte du code douteux en qualité, pour moi ça passe pas en review

Louis-htmlcss commented 6 days ago

est ce que c'est possible d'avoir un screenshot de ce que tu as changé avant et apres stp

Pour l'instant j'ai que ça car j'ai pas mon ordi pour lancer expo

ecnivtwelve commented 5 days ago

Pour l'instant j'ai que ça car j'ai pas mon ordi pour lancer expo

c'est pas circulaire ?

Louis-htmlcss commented 5 days ago

Pour l'instant j'ai que ça car j'ai pas mon ordi pour lancer expo

c'est pas circulaire ?

c'est ovale pour moi 🩻

PapillonApp / Papillon

refactor(LastGrade): Amélioration de l'affichage de l'emoji avec un cadre circulaire #202

🚀 Nouvelle Pull Request

Informations importantes

Checklist d'avant pull request

Changelogs proposés