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

fluid feature #408

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

fluid feature #408

wants to merge 3 commits into from

Conversation

cedric07
Copy link
Contributor

@cedric07 cedric07 commented May 21, 2024

  • Ajout de la notion de fluidité
  • Passage des gutter en variable CSS et ajout d'un gutter mobile récurent. On garde les variables SASS obligatoire pour le breakpoint, si pas nécessaire, on peut supprimer
  • Refacto container default / wide / full car des soucis entre BO et FO sur le responsive. On est maintenant ISO des deux côtés

Fluidité :

Enregistrement.de.l.ecran.2024-05-21.a.15.48.47.mov

Container FO :

Enregistrement.de.l.ecran.2024-05-21.a.14.03.26.mov

Container BO avant :

Enregistrement.de.l.ecran.2024-05-21.a.14.00.33.mov

Container BO maintenant :

Enregistrement.de.l.ecran.2024-05-21.a.14.01.26.mov

Copy link
Contributor

Choose a reason for hiding this comment

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

@cedric07 on part pas sur du fluide par défaut ?

ptesei
ptesei previously approved these changes Sep 2, 2024
@francoistibo
Copy link
Contributor

francoistibo commented Sep 9, 2024

@ptesei juste considération technique qui nous évite des breakpoints
@MarieComet à voir avec la DA pour le prendre en compte entièrement ?
@cedric07 démo d'Arendt pour bien comprendre

A valider le 23.09

@francoistibo
Copy link
Contributor

@n-langle : rajouter la notion de breakpoints dans le cadre du projet ODDO

@francoistibo
Copy link
Contributor

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants