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

[BUGFIX] Affichage de la vidéo au scroll sur la page d'accueil #317

Merged
merged 1 commit into from
Nov 2, 2021

Conversation

yannbertrand
Copy link
Member

🦄 Problème

La vidéo de la page d'accueil semblait passer sous le contenu du site.
image
image

🤖 Solution

Déplacement de l'image de fond de la PageBanner pour ne pas avoir besoin du clip-path. L'image gère elle même l'effet arrondi.

🌈 Remarques

Le soucis venait du clip-path sur le PageBanner. On ne met la couleur de fond bleu seulement si il n'y a pas d'image précisée.

💯 Pour tester

  1. Lancer la vidéo et scroller
  2. Vérifier que l'image du PageBanner sont correctes sur Pix pro/Pix Org

@yannbertrand yannbertrand added 👀 Tech Review Needed 👀 Func Review Needed team-evaluation PR relatives à l'expérience d'évaluation cross-team Toutes les équipes de dev labels Oct 29, 2021
@pix-service
Copy link

I'm deploying this PR to these urls:

Please check it out!

Copy link
Member

@MelanieMEB MelanieMEB left a comment

Choose a reason for hiding this comment

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

🎊

@pix-service-auto-merge pix-service-auto-merge force-pushed the fix-video-display-page-banner branch from 4d4c145 to 9c04adb Compare November 2, 2021 09:03
@pix-service-auto-merge pix-service-auto-merge merged commit 7727c7d into dev Nov 2, 2021
@pix-service-auto-merge pix-service-auto-merge deleted the fix-video-display-page-banner branch November 2, 2021 09:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cross-team Toutes les équipes de dev 🚀 Ready to Merge team-evaluation PR relatives à l'expérience d'évaluation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants