Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add widget to display the latest obtained grade #179

Merged
merged 12 commits into from
Sep 7, 2024
Merged

Add widget to display the latest obtained grade #179

merged 12 commits into from
Sep 7, 2024

Conversation

Crazzynel
Copy link
Contributor

@Crazzynel Crazzynel commented Sep 7, 2024 •
edited
Loading

🚀 Nouvelle Pull Request

Checklist d'avant pull request

  • Vous avez testé de build le projet avec vos modifications et ce build a réussi
  • Vous respectez les conventions de codage et de nommage du projet
  • Vous utilisez la tabulation pour l'indentation afin de maintenir un code lisible
  • Cette pull request n'est pas un duplicata d'une autre
  • Cette pull request est prête à être revue (review) et fusionnée (merge)
  • Il n'y a pas de TODO (aka des annotations pour du code manquant) dans vos modifications
  • Il n'y a pas d'erreurs de langue dans votre code (grammaire, vocabulaire, conjugaison, orthographe)
  • Les détails des changements ont été décrits ci-dessous
  • 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

  • Updated the LastGradeWidget component to always display the subject name instead of the grade comment, regardless of the situation.
  • Removed color coding for grades, using a single black color for the grade display, except for the grading scale where the original color is maintained.

Informations supplémentaires

  • La capture d'écran montrera l'apparence actuelle du widget après les modifications.

Capture d'écran de la modification

Screenshot_2024-09-07-18-53-53-946_xyz getpapillon app-edit

Crazzynel and others added 6 commits September 7, 2024 15:40
@Crazzynel
ADD: Addition of the "Last Grade" Widget
dc9fa48 
On the homepage, the widget now displays the last obtained grade.
@Crazzynel
FIX
8dd9cd0 
Fixed an issue preventing correct text display and adjusted the grade according to the professor's grading scale. Added color coding based on the grade.
@Crazzynel
Addition
0ceafac 
Added comments to facilitate understanding (in English).
@Crazzynel
Icon and Display Change
3b7ba0e 
Replaced the widget icon from a book (Book) to a pie chart (PieChart).

In case of unavailable grade comments (e.g., Spanish: Semester 1 Evaluation), the subject name is displayed immediately.
@Crazzynel
Merge branch 'PapillonApp:main' into new/widget-last-grade
44a3a97
@Crazzynel
Fix: Set note color to black and maintain original color for grading …
7b9319b 
…scale

- Updated the `LastGradeWidget` component to always display the note value in black.
- Maintained the original color for the grading scale.
- Adjusted styles to ensure consistent appearance across different note percentages.
@tom-theret
Copy link
Contributor

C'est très joli mais pourquoi une différence de circle background entre le Widget et le la vue classique ?

@Crazzynel
Copy link
Contributor Author

C'est très joli mais pourquoi une différence de circle background entre le Widget et le la vue classique ?

Genre au niveau de la taille du widget ? Je devrais uniformiser le tout ?

Jsp j'ai fait comme j'ai senti honnêtement. Si il faut que j'uniformise dis le moi !

@tom-theret
Copy link
Contributor

image

Tu vois la couleur derrière l'ordinateur n'est pas la même

@Crazzynel
Copy link
Contributor Author

Crazzynel commented Sep 7, 2024 •
edited
Loading

Ah ok çaaaa...
Je n'avais pas compris... Je vais voir si je peux ajouter ça merciii beaucoup @tom-theret !

raphckrman
raphckrman reviewed Sep 7, 2024
View reviewed changes
src/widgets/Components/LastGrade.tsx Show resolved Hide resolved
src/widgets/Components/LastGrade.tsx Outdated Show resolved Hide resolved
src/widgets/Components/LastGrade.tsx Outdated Show resolved Hide resolved
src/widgets/Components/LastGrade.tsx Outdated Show resolved Hide resolved
@Crazzynel @raphckrman
Apply suggestions from code review
128999f 
Updating. Correction pour compatibilité avec le mode sombre et pour garantir une logique complète même sans l'utilisation des pourcentages de note

Co-authored-by: Raphaël <[email protected]>
@Crazzynel Crazzynel requested a review from raphckrman September 7, 2024 18:12
@Crazzynel
Copy link
Contributor Author

@tom-theret corrigé !

Merci @raphckrman pour l'aide !

raphckrman
raphckrman approved these changes Sep 7, 2024
View reviewed changes
Copy link
Contributor

@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.

LGTM

@Crazzynel
Copy link
Contributor Author

Et je pense que vu que ça touche à l'UI il faudrait aussi une review de @ecnivtwelve en plus de tout le monde... jsp...

@yannouuuu yannouuuu added ✨ enhancement New feature or request 🎨 interface Element visible to the user 🚸 user experience UX related issues labels Sep 7, 2024
@Crazzynel
Update by review
5e2a3c8 
Mise à jour de l'affichage (by @Gabriel29306)
Gabriel29306
Gabriel29306 approved these changes Sep 7, 2024
View reviewed changes
Copy link
Contributor

@Gabriel29306 Gabriel29306 left a comment

Choose a reason for hiding this comment

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

LTGM

@ecnivtwelve
Copy link
Contributor

J'ai apporté quelques modifs pour améliorer l'affichage du widget, sinon c'est très bien !

@ecnivtwelve ecnivtwelve merged commit c1f5004 into PapillonApp:main Sep 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ecnivtwelve ecnivtwelve ecnivtwelve approved these changes

@raphckrman raphckrman raphckrman approved these changes

@Gabriel29306 Gabriel29306 Gabriel29306 approved these changes

@toi-et-moi toi-et-moi toi-et-moi approved these changes

Assignees
No one assigned
Labels
✨ enhancement New feature or request 🎨 interface Element visible to the user 🚸 user experience UX related issues
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
7 participants
@Crazzynel @tom-theret @ecnivtwelve @raphckrman @Gabriel29306 @toi-et-moi @yannouuuu