Development #51802
ajouter class .has-pwa-navigation sur le body
0%
Description
On peut activer le mode PWA sur un thème sans pour autant utilisé la nav PWA
Donc ajouter une class CSS `has-pwa-navigation` uniquement si {% pwa_navigation %} ne retourne pas un string vide permet :
- détecter en CSS si la nav pwa est utilisée.
- Permet de déplacer la nav dans le template sans casser le margin-bottom ajouté au footer
.has-pwa-navigation footer
étant plus solide que
.pwa-navigation ~ footer
Fichiers
Demandes liées
Révisions associées
Historique
Mis à jour par Thomas Jund (congés, retour le 29/04) il y a environ 3 ans
- Fichier 0001-templates-add-conditionnal-has-pwa-navigation-CSS-cl.patch 0001-templates-add-conditionnal-has-pwa-navigation-CSS-cl.patch ajouté
- Fichier 0001-pwa-nav-template-no-newline-at-end-of-file-51802.patch 0001-pwa-nav-template-no-newline-at-end-of-file-51802.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Petit patch nécessaire sur le template combo de la nav pwa pour supprimer le dernier retour chariot en fin de ficher (après le endif) pour retourner un vrai empty string si pas d'item.
Il y a peut-être plus solide / plus simple comme patch que d'associer le tag pwa_navigation à une variable, mais pas trouvé.
Mis à jour par Thomas Jund (congés, retour le 29/04) il y a environ 3 ans
- Fichier 0001-templates-add-conditionnal-has-pwa-navigation-CSS-cl.patch 0001-templates-add-conditionnal-has-pwa-navigation-CSS-cl.patch ajouté
La class `.pwa` ajouté par #34430 ne fait donc pas son job.
Je la supprime pour la remplacer par cette nouvelle class plus solide.
Mis à jour par Thomas Jund (congés, retour le 29/04) il y a environ 3 ans
oui, c'est l'id du patch.
id du ticket : #32974
Mis à jour par Serghei Mihai il y a environ 3 ans
- Statut changé de Solution proposée à En cours
Pourquoi ?
{% if pwa_navigation %}has-pwa-navigation{% endif %}
{% if pwa_display %}
ne fait pas le boulot?
Mis à jour par Thomas Jund (congés, retour le 29/04) il y a environ 3 ans
Le patch #32974 vérifie juste si la variable `"pwa_display": "standalone"`est posée.
Donc que le mode PWA est activé.
Mais cela ne veut pas dire que la nav PWA est utilisée. On peut très bien activé le mode PWA sans utiliser la nav.
L'idée du patch #32974 était de poser une class sur le body pour pouvoir conditionner si la nav PWA est présente ou pas, et en fait il ne vérifie pas cela.
Mis à jour par Thomas Jund (congés, retour le 29/04) il y a environ 3 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit combo
commit 3dd0b172d835f6575978ede58b758e4c21a66d40 (HEAD -> main, origin/main)
Author: Thomas JUND <tjund@entrouvert.com>
Date: Mon Mar 8 17:03:51 2021 +0100
pwa-nav template: no newline at end of file (#51802)
to return an empty string if no item
commit publik-base-theme
commit 307fab949654a5808f92e1ca105cbd72f7ff4a81 (HEAD -> main, origin/main)
Author: Thomas JUND <tjund@entrouvert.com>
Date: Mon Mar 8 16:57:06 2021 +0100
templates: add conditionnal 'has-pwa-navigation' CSS class on body (#51802)
Mis à jour par Frédéric Péters il y a environ 3 ans
- Statut changé de Résolu (à déployer) à Solution déployée
Mis à jour par Frédéric Péters il y a 26 jours
- Lié à Bug #88994: toodego, visibilité du pied de page en mobile ajouté
templates: add conditionnal 'has-pwa-navigation' CSS class on body (#51802)