Projet

Général

Profil

Bug #61082

class .has-pwa-navigation est ajouté même si aucune entrées de menu existe

Ajouté par Thomas Jund (congés, retour le 29/04) il y a environ 2 ans. Mis à jour il y a environ 2 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Frédéric Péters
Version cible:
-
Début:
25 janvier 2022
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

`{{ pwa_navigation|lenght }}` return "1" car le template retourne un saut de ligne.

Fichiers

0001-template-never-add-.has-pwa-navigation-class-if-navi.patch (1,07 ko) 0001-template-never-add-.has-pwa-navigation-class-if-navi.patch Thomas Jund (congés, retour le 29/04), 25 janvier 2022 17:28
0001-pwa-strip-navigation-to-avoid-trailing-whitespaces-f.patch (882 octets) 0001-pwa-strip-navigation-to-avoid-trailing-whitespaces-f.patch Frédéric Péters, 25 janvier 2022 17:43
0001-pwa-strip-navigation-to-avoid-trailing-whitespaces-f.patch (1,06 ko) 0001-pwa-strip-navigation-to-avoid-trailing-whitespaces-f.patch Frédéric Péters, 25 janvier 2022 18:42

Révisions associées

Révision bcdb70ff (diff)
Ajouté par Frédéric Péters il y a environ 2 ans

pwa: strip navigation to avoid trailing whitespaces from template (#61082)

Historique

#1

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

  • Tracker changé de Support à Bug
#2

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

Vu sur le thème de Caen ou la marge sous le footer est conservée alors que la nav PWA n'est pas utilisée.

#3

Mis à jour par Frédéric Péters il y a environ 2 ans

  • Statut changé de Solution proposée à En cours
  • Assigné à changé de Thomas Jund (congés, retour le 29/04) à Frédéric Péters
  • Patch proposed changé de Oui à Non

Euh le patch il dit "si la navigation pwa est une chaine vide alors has-pwa-navigation". Ça n'est je pense pas ce qui est souhaité.

Et ça sera plus propre de laisser la condition simple; plutôt assurer le côté combo et stripper.

#4

Mis à jour par Frédéric Péters il y a environ 2 ans

  • Projet changé de Intégrations graphiques Publik à Combo
#5

Mis à jour par Frédéric Péters il y a environ 2 ans

#6

Mis à jour par Serghei Mihai il y a environ 2 ans

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

J'étais en train d'écrire la même chose que Fréd.
Go pour le patch Combo.

#7

Mis à jour par Serghei Mihai il y a environ 2 ans

  • Statut changé de Solution validée à En cours

Je rétropédale: avec l'ajout du strip le contenu n'est pas "safe".

#8

Mis à jour par Frédéric Péters il y a environ 2 ans

Oui très bien il y a des tests pour ça.

#10

Mis à jour par Serghei Mihai il y a environ 2 ans

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

Mis à jour par Frédéric Péters il y a environ 2 ans

  • Statut changé de Solution validée à Résolu (à déployer)
commit bcdb70fff919f7cd6e280e760575db0e7075ac63
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Tue Jan 25 17:43:27 2022 +0100

    pwa: strip navigation to avoid trailing whitespaces from template (#61082)
#12

Mis à jour par Transition automatique il y a environ 2 ans

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

Mis à jour par Transition automatique il y a environ 2 ans

Automatic expiration

Formats disponibles : Atom PDF