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.

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

[FEATURE] Ajout d'un bouton reload pour les embed (PIX-15418) #10784

Merged
merged 2 commits into from
Dec 16, 2024

Conversation

rouxxi
Copy link
Contributor

@rouxxi rouxxi commented Dec 11, 2024

🎄 Problème

Les utilisateurs ne peuvent pas reload l'embed sans recharger la page.

🎁 Proposition

Ajout d'un bouton "Recommencer" en dessous des embeds avant validation.

🧦 Remarques

RAS

🎅 Pour tester

Ce boutton ne doit pas s'afficher sur des carrousel / QCU_Image / etc...

Possible de tester avec un simulateur qui doit avoir le bouton de reload => https://junior-pr10784.review.pix.fr/challenges/challenge1JkmTl7OQTiW4z/preview

Lors de la validation il faut que le boutton soit caché

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

@aurelie-crouillebois aurelie-crouillebois changed the title [FEATURE] Ajout d'un boutton reload pour les embed (PIX-15418) [FEATURE] Ajout d'un bouton reload pour les embed (PIX-15418) Dec 13, 2024
@rouxxi rouxxi force-pushed the pix-15418-add-restart-to-junior-embed branch from e8c8a01 to 215597c Compare December 13, 2024 11:32
@mcampourcy
Copy link
Contributor

Test func OK ✅

@mcampourcy mcampourcy added Func Review OK PO validated functionally the PR and removed 👀 Func Review Needed labels Dec 13, 2024
@pierrepougetpix
Copy link

nickel mais peut-on ajouter de la marge entre l'icone et le texte en appliquant le style tertiary du button de Pix UI ?(https://ui.pix.fr/?path=/docs/actions-button--docs)

@pix-service-auto-merge pix-service-auto-merge force-pushed the pix-15418-add-restart-to-junior-embed branch from 40931a1 to e39959c Compare December 16, 2024 15:32
@pix-service-auto-merge pix-service-auto-merge merged commit de88be7 into dev Dec 16, 2024
7 of 8 checks passed
@pix-service-auto-merge pix-service-auto-merge deleted the pix-15418-add-restart-to-junior-embed branch December 16, 2024 15:38
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.

6 participants