Produits Entr'ouvert » Intégrations graphiques Publik

Avec adaptations aux intégrations Metz Métropole et Quimper, pour donner une idée, je dois encore y ajouter de la documentation, etc.

Voilà avec le tout documenté.

Dans templates/combo/page_template.html, cette partie :

{% if pwa_display == "standalone" or pwa_display == "fullscreen" %}
{% pwa_navigation %}
{% endif %}

me semble faire doublon avec le code de pwa_navigation :

@register.simple_tag(takes_context=True)
def pwa_navigation(context):
    if settings.TEMPLATE_VARS.get('pwa_display') not in ('standalone', 'fullscreen'):
        return ''
    ...

Je serais pour retirer celle du templatetags, mais bon, c'est un autre patch dans combo à faire, je sais pas trop. On peut aussi garder les deux, ça change rien au final. Je valide la proposition donc, tu décides juste avant de pousser...

(Rien d'autre à dire ; pour le css c'est de toute façon nécessaire qu'il tourne en recette pour être "vraiment" validé)

J'avais oublié que je l'avais ajouté côté combo et du coup dupliqué ici... À mon avis plutôt le retirer côté combo.

commit ac6f90e310be3634043f5154fad618fe770167c9
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Fri Dec 28 10:14:00 2018 +0100

    general: add support for new PWA navigation (#29380)

Et parce que sass refuse l'utilisation d'@extend appliqués à des media queries différentes, voici la transformation pour utiliser des mixin à la place.

commit 51f4df9446d75a5cd5f5afa14527a91720d77e99
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Thu Jan 3 15:45:09 2019 +0100

    scss: use a mixin for pwa navigation bar (#29380)