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

Corrige le style du pied-de-page pour qu'il ne dépasse plus #6395

Merged
merged 3 commits into from
Oct 8, 2022

Conversation

Arnaud-D
Copy link
Contributor

@Arnaud-D Arnaud-D commented Oct 5, 2022

Fix #6146.

Le pied de page se réduit désormais correctement sans causer une bande blanche sur la droite pour certaines t'ailles d'écran.

Contrôle qualité

Réduire une page progressivement et voir que le pied de page est toujours bien affiché.

Note: Cela marche parfaitement sur certaines pages (forum par exemple), mais pas sur d'autres (bibliothèque, page d'accueil) où il reste une bande blanche bien que le pied de page soit bien en ordre. C'est un souci indépendant lié aux colonnes de vignettes, qui est hors du périmètre de cette PR.

@Arnaud-D Arnaud-D added the C-Front Concerne l'interface du site label Oct 5, 2022
@coveralls
Copy link

coveralls commented Oct 5, 2022

Coverage Status

Coverage remained the same at 88.303% when pulling 0a884a9 on Arnaud-D:coupe-ce-qui-depasse-1 into 08d06b1 on zestedesavoir:dev.

Copy link
Member

@Situphen Situphen left a comment

Choose a reason for hiding this comment

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

Ta solution permet en effet d'éviter que le pied-de-page ne dépasse à droite à certains moments, mais en effet secondaire les boutons des réseaux sociaux ne sont plus centrés au milieu. C'est une histoire de goûts, mais j'ai trouvé une solution qui permet de garder ces boutons centrés au milieu jusqu'à ce que ne ce soit plus possible à cause de l'espace disponible.

Tous les éléments (gauche, centre et droite) ont flex-shrink: 0 ce qui les empêche de rétrécir par rapport à leur taille initiale. L'élément central a flex-grow: 1 tandis que ceux de gauche et de droite ont flex-grow: 5 donc ils prennent chacun 5 fois plus de place que l'élément central quand c'est possible (ce qui permet de garder cet élément centré). Enfin, l'élément central a flex-basis: auto ce qui lui confère une petite marge et évite que les textes soient collés quand on rétrécit la fenêtre ; les deux autres ont flex-basis: 0.

Voici le diff à appliquer sur ta branche :

diff --git a/assets/scss/layout/_footer.scss b/assets/scss/layout/_footer.scss
index 2bd03b776..9cf4b8615 100644
--- a/assets/scss/layout/_footer.scss
+++ b/assets/scss/layout/_footer.scss
@@ -13,9 +13,9 @@
     }
 
     .copyright, .links {
-        flex-shrink: 1;
-        flex-grow: 1;
-        min-width: 0;
+        flex-shrink: 0;
+        flex-grow: 5;
+        flex-basis: 0;
     }
 
     .copyright {
@@ -63,7 +63,7 @@
         }
 
         &.social {
-            flex-grow: 0;
+            flex-grow: 1;
             flex-shrink: 0;
             flex-basis: auto;
             text-align: center;

@Arnaud-D Arnaud-D force-pushed the coupe-ce-qui-depasse-1 branch from 9f0b95d to 0c61ee2 Compare October 7, 2022 19:04
@Arnaud-D
Copy link
Contributor Author

Arnaud-D commented Oct 7, 2022

J'ai fait ta modif. Perso, ça m'est égal que ça soit centré, je ne trouve pas que ça change grand chose au design. Voire même ça fait moins équilibré vu que la liste de liens est très longue.

@Arnaud-D Arnaud-D added the hacktoberfest-accepted Pull request approuvée pour le Hacktoberfest label Oct 7, 2022
Copy link
Member

@Situphen Situphen left a comment

Choose a reason for hiding this comment

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

QA OK

@Situphen Situphen enabled auto-merge (squash) October 8, 2022 10:18
@Situphen Situphen merged commit 893bc33 into zestedesavoir:dev Oct 8, 2022
@Arnaud-D Arnaud-D deleted the coupe-ce-qui-depasse-1 branch October 9, 2022 08:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-Front Concerne l'interface du site hacktoberfest-accepted Pull request approuvée pour le Hacktoberfest
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Bande blanche et barre de défilement horizontale dans un cas particulier
3 participants