Skip to content

Conversation

@Grise3
Copy link
Contributor

@Grise3 Grise3 commented Jan 27, 2026

Contribution

Règles de contribution

Caution

Afin de garantir une application stable et pérenne dans le temps, nous t'invitons à vérifier que tu as bien respecté les règles de contribution. Sans cela, ta Pull Request ne pourra pas être examinée.

  • Cette Pull Request porte sur une seule fonctionnalité ou un seul correctif.
  • Cette Pull Request n'est pas faite essentiellement avec de l'IA.
  • Pour tout changement majeur, j’ai créé une issue afin d’échanger avec les mainteneurs de Papillon sur la meilleure façon de l’intégrer.
  • Ma Pull Request respecte les conventions Conventional Commits et Conventional Branch ainsi que les conventions de codage de l'application.
  • J’ai testé mes modifications sur iOS et Android, et l’application fonctionne correctement.
  • J’emploie un langage informel, clair et concis dans mes messages.
  • J’ai documenté mes changements de manière appropriée, soit dans la description de la Pull Request, soit dans le GitBook.
  • J’ai ajouté les traductions nécessaires dans au moins un fichier de langue.

Résumé des changements

Ajout d'un texte indiquand si une note est optionnelle ou bonus.

Capture(s) d'écran

image

@raphckrman
Copy link
Member

Hello, ça rend comment si c'est une note bonus et que l'élève a la meilleure note ?

@Grise3
Copy link
Contributor Author

Grise3 commented Jan 27, 2026

Hello, ça rend comment si c'est une note bonus et que l'élève a la meilleure note ?

Aucune idée... Je vais faire le test de suite !

Copy link
Member

@raphckrman raphckrman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Très clean pour l'instant, si tu peux regarder les deux petits commentaires ce serait parfrait

@Grise3
Copy link
Contributor Author

Grise3 commented Jan 27, 2026

@raphckrman :

image

@Grise3

This comment was marked as resolved.

@Grise3
Copy link
Contributor Author

Grise3 commented Jan 27, 2026

c'est fait

Copy link
Member

@ryzenixx ryzenixx left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hello ! Le badge de la meilleure note doit rester exclusif, appliquer la même UX a une autre information casse la chaine de récompense apportée par le badge de la note :/

@PapillonApp PapillonApp deleted a comment from raphckrman Jan 27, 2026
@raphckrman raphckrman requested a review from ryzenixx January 27, 2026 22:50
@ryzenixx ryzenixx requested a review from raphckrman January 27, 2026 22:50
@Grise3
Copy link
Contributor Author

Grise3 commented Jan 27, 2026

Que suggérez-vous ?
En ce qui me concerne je ne vois pas ce qui casse la chaîne de récompenses. Et puis... Le "meilleur note de votre groupe" sera toujours en haut, et, mis en avant sur la page principale des notes, comme sur l'image ci-dessous (contrairement au note bonus/optionnelle)

Screenshot_20260127-225501_Papillon_1.png

@ecnivtwelve
Copy link
Contributor

C'est une bonne idée de l'afficher mais pas forcément en premier plan pour laisser cet espace aux badges exceptionnels (du type "Meilleure Note").

Je pense qu'au même endroit mais sans couleur (en texte secondaire – de manière plus discrète) remplirait mieux le rôle de cette information

@Grise3
Copy link
Contributor Author

Grise3 commented Jan 28, 2026

@ecnivtwelve comme ceci, ça serait mieux ?
Ou bien il faut que ça se remarque moins ?

Screenshot_20260128-091857_Papillon_1.png

@ecnivtwelve
Copy link
Contributor

ecnivtwelve commented Jan 28, 2026

@ecnivtwelve comme ceci, ça serait mieux ? Ou bien il faut que ça se remarque moins ?

Faudra demander à @tom-things il aura le dernier mot ;)

@Grise3
Copy link
Contributor Author

Grise3 commented Jan 28, 2026

Okay, en attente de sa réponse alors :)

@Grise3
Copy link
Contributor Author

Grise3 commented Feb 8, 2026

Voici donc ce à quoi ça ressemble maintenant :

image

@tom-things
Copy link
Contributor

C'est vraiment bien mieux comme ça :)

@tom-things tom-things self-requested a review February 8, 2026 12:19
@Grise3
Copy link
Contributor Author

Grise3 commented Feb 8, 2026

Je suis d'accord, c'est bien mieux comme ça !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants