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

[FEATURE] Utiliser PixIcon dans Pix App - Partie 1 (PIX-14775). #10632

Open
wants to merge 7 commits into
base: dev
Choose a base branch
from
Open

[FEATURE] Utiliser PixIcon dans Pix App - Partie 1 (PIX-14775). #10632

wants to merge 7 commits into from
+297 −262

Conversation

AndreiaPena
Copy link
Member

@AndreiaPena AndreiaPena commented Nov 25, 2024 •
edited
Loading

🍂 Problème

Le composant PixIcon existe et les FaIcon peuvent désormais être remplacés.

🌰 Proposition

Utiliser PixIcon

🎃 Remarques

Lorsque arrive à une page de step lors d'un passage d'épreuve, une phrase présentait un contraste trop faible, c'est corrigé

ok

J'ai aussi fixé la page oups avec son bouton carré (https://app-pr10632.review.pix.fr//campagnes/EVALBADGE/evaluation/resultats)

Capture d’écran 2024-11-25 à 19 29 29 Capture d’écran 2024-11-25 à 19 31 16

🪵 Pour tester

Note

Coté épreuves :

Étapes Screens
Signaler une épreuve (voir screen) et constater que les icônes warning et flag apparaîssent Capture d’écran 2024-11-25 à 18 08 42
Signaler une épreuve (voir screen) et constater que les icônes stylo apparaît Capture d’écran 2024-11-26 à 17 20 49
Sur une épreuve avec embed, constater que l'icône refresh apparaît ok
Sur une épreuve avec focus, défocus la question et constater que l'icône info apparaît Capture d’écran 2024-11-25 à 18 20 15
(Je ne sais pas reproduire celui-ci hors local, mais cette icône s'affiche lorsqu'une image ou un embed n'apparaît pas) Capture d’écran 2024-11-25 à 18 25 05
Dans une épreuve, revenir à l'épreuve précédente et constater que l'icône lock apparaît Capture d’écran 2024-11-26 à 18 22 47
Dans un checkpoint, constater que les icônes de succès ou d'erreur s'affichent ainsi que l'icône pour quitter le passage des questions Capture d’écran 2024-11-26 à 18 25 07
Tester avec une épreuve timé -

Note

Coté Accès :

https://app-pr10632.review.pix.fr/recuperer-mon-compte Constater que l'icône info apparaît sur le champ ok
Dans le menu utilisateur (desktop) constater que l'icône power apparaît Capture d’écran 2024-11-25 à 15 21 42
Dans la sidebar (mobile) constater que l'icône power apparaît -
Lors d'une inscription à Pix App, constater que les icône de validation et d'erreur pour le mot de passe apparaîssent Capture d’écran 2024-11-26 à 18 29 15

Note

Coté modulix :

Sur un module avec un bouton de téléchargement, constater la présence de l'icône fenêtre Capture d’écran 2024-11-26 à 18 01 44
Sur le récap, constater la présence de l'icône de validation Capture d’écran 2024-11-26 à 18 01 14
https://app-pr10632.review.pix.fr/plan-du-site Constater que les icônes externe s'affichent ok

@AndreiaPena AndreiaPena added Development in progress cross-team Toutes les équipes de dev labels Nov 25, 2024
@AndreiaPena AndreiaPena self-assigned this Nov 25, 2024
@pix-bot-github
Copy link

Une fois les applications déployées, elles seront accessibles via les liens suivants :

Les variables d'environnement seront accessibles via les liens suivants :

@AndreiaPena AndreiaPena force-pushed the pix-14775-use-pix-icon-on-pix-app branch 5 times, most recently from e3c692f to 08cb5e5 Compare November 26, 2024 17:29
@AndreiaPena AndreiaPena marked this pull request as ready for review November 26, 2024 17:30
@AndreiaPena AndreiaPena requested review from a team as code owners November 26, 2024 17:30
@AndreiaPena AndreiaPena changed the title [FEATURE] Utiliser PixIcon dans Pix App (PIX-14775). [FEATURE] Utiliser PixIcon dans Pix App - Partie 1 (PIX-14775). Nov 26, 2024
@AndreiaPena AndreiaPena force-pushed the pix-14775-use-pix-icon-on-pix-app branch 4 times, most recently from 3640417 to dde5d6e Compare November 27, 2024 08:33
bpetetot
bpetetot approved these changes Nov 27, 2024
View reviewed changes
Copy link
Contributor

@bpetetot bpetetot left a comment

Choose a reason for hiding this comment

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

👍 Review OK et testé côté Accès. Merci !

xav-car
xav-car reviewed Nov 27, 2024
View reviewed changes
high-level-tests/e2e/cypress/support/step_definitions/assessment.js
@@ -35,20 +35,17 @@ Then(`je vois la page de résultats`, () => {
Then(`j'ai passé à {string}`, (challenge) => {
cy.contains(".result-item", challenge)
.find(".result-item__icon svg")
.should("have.class", "fa-circle-xmark")
.and("have.class", "result-item__icon--grey");
.should("have.class", "result-item__icon--grey");
Copy link
Contributor

Choose a reason for hiding this comment

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

je me pose la question de l'utilité de cette recherche dans un test e2e 🤔 . mais bon c'était déjà là avant. 😅

@Libouk
Copy link
Member

Libouk commented Nov 28, 2024

  • "Signaler une épreuve (voir screen) et constater que les icônes warning et flag apparaîssent" c'est warning et reload qui s'affichent mais c'est bon il me semble.
    Sinon tout est ok pour les épreuves !

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

@dianeCdrPix dianeCdrPix dianeCdrPix approved these changes

@Libouk Libouk Libouk approved these changes

@xav-car xav-car xav-car approved these changes

@bpetetot bpetetot bpetetot approved these changes

Assignees

@AndreiaPena AndreiaPena

Labels
cross-team Toutes les équipes de dev 🚀 Ready to Merge
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@AndreiaPena @pix-bot-github @Libouk @bpetetot @xav-car @dianeCdrPix