Bug #44255
pwa-botom-bar: propriété height en doublon
Début:
19 juin 2020
Echéance:
% réalisé:
0%
Temps estimé:
Patch proposed:
Oui
Planning:
Non
Description
Vu en bossant sur le thème de Villeneuve d'ascq, sur lequelle j'affiche la nav PWA en static en mode desktop et à laquelle j'ai ajouté des bordures top et bottom.
height: $nav-mobile-bottom-bar-height;
est déclaré 2x : une fois sur le composant `pwa-navigation` ou `nav` et également sur les liens.
Comme le core est maintenant par défaut en `box-sizing: border-box`, avec une variable `$nav-mobile-bottom-bar-height: 64px`, si on ajoute un border au composant `border: 10px solid`
le content du composant ne fera plus que 54px de hauteur. Et le lien toujours 64px, il va donc déborder de 10 px du composant.
Déclarer la hauteur sur le lien est suffisant pour gérer la hauteur du composant.
Fichiers
Révisions associées
Historique
Mis à jour par Thomas Jund (congés, retour le 29/04) il y a presque 4 ans
- Fichier 0001-css-no-longer-declare-height-on-pwa-navigation-compo.patch 0001-css-no-longer-declare-height-on-pwa-navigation-compo.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Frédéric Péters il y a presque 4 ans
- Statut changé de Solution proposée à Solution validée
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 7238a435ccc604004de6d0af95aee46b942ae4f1
Author: Thomas JUND <tjund@entrouvert.com>
Date: Fri Jun 19 10:52:45 2020 +0200
css: no longer declare height on pwa-navigation component (#44255)
height is already declared on links
Mis à jour par Frédéric Péters il y a presque 4 ans
- Statut changé de Résolu (à déployer) à Solution déployée
css: no longer declare height on pwa-navigation component (#44255)
height is already declared on links