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] Passage > Mettre la Beta Banner au dessus de la navbar (PIX-15698)(PIX-15832) #10898

Open
wants to merge 1 commit into
base: dev
Choose a base branch
from

Conversation

dianeCdrPix
Copy link
Contributor

@dianeCdrPix dianeCdrPix commented Dec 24, 2024

🎄 Problème

Dans le passage d'un module, la bannière Beta est affichée sous la navbar, en largeur réduite. On veut garder une cohérence avec l'affichage dans details et recap.

🎁 Proposition

Afficher la betaBanner au dessus de la navbar d'un module.

🧦 Remarques

  • Choix de ne pas ancrer le beta-banner, comme dans details.js
  • Pour l'instant, on continue à utiliser le composant PixNotificationAlert, en attendant le nouveau variant Pix-UI pour PixBannerAlert.

🎅 Pour tester

  • Ouvrir un module dans Pix
  • Commencer le module
  • Vérifier que la bannière beta s'affiche en haut de page, au dessus de la NavBar

@dianeCdrPix dianeCdrPix self-assigned this Dec 24, 2024
@dianeCdrPix dianeCdrPix requested a review from a team as a code owner December 24, 2024 13:23
@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 :

@dianeCdrPix dianeCdrPix changed the title Passage > Mettre la Beta Banner au dessus de la navbar (PIX-15698)(PIX-15832) [FEATURE] Passage > Mettre la Beta Banner au dessus de la navbar (PIX-15698)(PIX-15832) Dec 24, 2024
@clemlatz clemlatz force-pushed the pix-15832-beta-banner-above-navbar-in-module branch from 86c4ff8 to cda0065 Compare January 2, 2025 10:20
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.

3 participants