Produits Entr'ouvert » Intégrations graphiques Publik

Border-radius

.gru-content div.link-list-cell,
.gru-content div.link-list-cell.has-asset-picture > div picture img,
.gru-content div.wcsformsofcategorycell,
.gru-content div.wcsformsofcategorycell.has-asset-picture > div picture img,
.gru-content div.wcsformsofcategorycell.has-asset-picture > div > h2:first-child {
    border-radius: 20px;
    @media screen and (max-width: $nav-mobile-limit) {
        width: 100%;
    }
}

C'est un peu compliqué d'ajouter un border-radius à une cellule sans passer par $cell-border-radius.
Dans ton inté, test un :hover sur un dernier item de liste, tu verras qu'il ne respecte pas le border-radius.
Il faut alors que tu ajoutes un `overflow: hidden` (si tu ne prévois pas d'afficher un élément hors de la boite c'est OK)

Breakpoint

Si tu n'utilises qu'un seul breakpoint, Il est préférable d'utiliser $mobile-limit ($nav-mobile-limit hérite par défaut de sa valeur)
$nav-mobile-limit, c'est plutôt pour dissocier le point de rupture entre nav mobile et desktop si la valeur de $mobile-limit n'est pas adéquate.

Me semble inutile

line 33

-        display: grid;

Il faudra limiter la nav à 4 items. Sinon ça passe à la ligne à 900px, valeur de ton breakpoint (mais c'est pareil sur site instit, donc…)

Un peu bizarre de donner la valeur `#footer-color; #000` et ensuite de surcharger en CSS avec

div#footer-wrapper {
    color: white;
}

Thomas Jund a écrit :

Border-radius

[...]

C'est un peu compliqué d'ajouter un border-radius à une cellule sans passer par $cell-border-radius.
Dans ton inté, test un :hover sur un dernier item de liste, tu verras qu'il ne respecte pas le border-radius.
Il faut alors que tu ajoutes un `overflow: hidden` (si tu ne prévois pas d'afficher un élément hors de la boite c'est OK)

En effet ça a l’air plus compliqué que ce que je pensais, et le overflow: hidden ne semble pas suffire. Je regarde comment corriger.

Pour toutes les autres remarques c’est intégré (visible dans la branche wip).

Benjamin Dauvergne a écrit :

j'ai supposé que la remarque sur le nombre d'éléments dans la nav n'avait pas de réalisation au niveau du patch, juste un truc à prendre en compte au niveau éditorial.

Oui c’est ce que j’ai compris aussi de la remarque de Thomas. Merci pour le ack.

Automatic expiration