Projet

Général

Profil

Development #45175

Development #44934: Plan amélioration layout des templates

Supprimer les marges gouttières des a2-blocks en contexte 2 colonnes et en mobile

Ajouté par Thomas Jund (congés, retour le 29/04) il y a presque 4 ans. Mis à jour il y a plus de 3 ans.

Statut:
Fermé
Priorité:
Normal
Version cible:
-
Début:
16 juillet 2020
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

Par défaut, un a2-block :
_cell.scss l.51

.a2-block {
  margin: 4em auto;
  max-width: 45em;
}

Mais ces marges sont modifiées pour correspondre aux gouttières (comme les autres cellules), lorsque les a2-blocks se trouvent dans un contexte 2 colonnes (assez rare à mon avis, mais bon).
_cell.scss l.83

div#left div.a2-block, div#right div.a2-block,
.gru-content div.cell,
.block {
    margin: 0px #{$columns-gutter / 2} 10px #{$columns-gutter / 2};
}

Pour .block et .cell, les marges gauches et droites sont ensuite supprimés en fonction de leur parent (#left ou #right) ainsi qu'en viewport mobile.
_cell.scss l.34

Ce n'est pas le cas des a2-block, qui sont alors non alignés et décalés.
Je propose d'appliquer ce même comportement aux a2-blocks également.


Fichiers

a2-block-margins-desktop.png (65,5 ko) a2-block-margins-desktop.png Thomas Jund (congés, retour le 29/04), 16 juillet 2020 13:11
a2-block-margins-mobile.png (41,1 ko) a2-block-margins-mobile.png Thomas Jund (congés, retour le 29/04), 16 juillet 2020 13:11
0001-css-remove-a2-block-margins-when-2-columns-mobile-vi.patch (1,63 ko) 0001-css-remove-a2-block-margins-when-2-columns-mobile-vi.patch Thomas Jund (congés, retour le 29/04), 16 juillet 2020 13:20

Révisions associées

Révision 0f2865de (diff)
Ajouté par Thomas Jund (congés, retour le 29/04) il y a presque 4 ans

css: remove a2-block margins when 2 columns & mobile viewport (#45175)

Historique

#1

Mis à jour par Thomas Jund (congés, retour le 29/04) il y a presque 4 ans

#2

Mis à jour par Serghei Mihai il y a presque 4 ans

  • Statut changé de Solution proposée à Solution validée
#3

Mis à jour par Thomas Jund (congés, retour le 29/04) il y a presque 4 ans

  • Statut changé de Solution validée à Résolu (à déployer)
commit 0f2865de97153a72b191e6b1f1879c885773675f (HEAD -> master, origin/master, origin/HEAD)
Author: Thomas JUND <tjund@entrouvert.com>
Date:   Mon Jul 6 16:49:55 2020 +0200

    css: remove a2-block margins when 2 columns & mobile viewport (#45175)
#4

Mis à jour par Frédéric Péters il y a plus de 3 ans

  • Statut changé de Résolu (à déployer) à Solution déployée

Formats disponibles : Atom PDF