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

[BREAKING] Utilisation de yield nommés dans le PixAppLayout (pix-15665) #792

Merged
merged 1 commit into from
Dec 13, 2024

Conversation

lionelB
Copy link
Member

@lionelB lionelB commented Dec 13, 2024

💥 BREAKING_CHANGES

Le Composant utilise désormais 3 yield nommés plutot qu'un seul yield

  • <:navigation>
  • <:main>
  • <:footer>

🎄 Problème

Le positionnement du footer se fait actuellement dans les applications est n'est pas toujours homogène.

🎁 Proposition

Afin d'améliorer l'homogénéité du design de nos applications, le positionnement du footer est géré par le composant AppLayout

🌟 Remarques

On a fait le choix de ne pas mettre de balise pour le main et le footer pour permettre de pouvoir rajouter une classe si besoin.
On profite de la PR pour rajouter une règle pour que le AppLayout prenne toute la hauteur disponible si le contenu tient dans le viewport.

🎅 Pour tester

Afficher la story du composant AppLayout

@lionelB lionelB self-assigned this Dec 13, 2024
@pix-bot-github
Copy link

Une fois l'application déployée, elle sera accessible à cette adresse https://ui-pr792.review.pix.fr
Les variables d'environnement seront accessibles sur scalingo https://dashboard.scalingo.com/apps/osc-fr1/pix-ui-review-pr792/environment

@pix-service-auto-merge pix-service-auto-merge merged commit ea3a034 into dev Dec 13, 2024
10 of 12 checks passed
@pix-service-auto-merge pix-service-auto-merge deleted the pix-15665/add-footer-to-pix-app-layout branch December 13, 2024 15:12
pix-service-auto-merge pushed a commit that referenced this pull request Dec 13, 2024
# [52.0.0](v51.6.1...v52.0.0) (2024-12-13)

### 💥 BREAKING CHANGE

- [#792](#792)  Utilisation de yield nommés dans le PixAppLayout (pix-15665)
@pix-service-auto-merge
Copy link
Contributor

🎉 This PR is included in version 52.0.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

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.

None yet

4 participants