Development #32974
affichage du bandeau de consentement de pose des cookies sous le menu PWA
0%
Description
Le texte n'est visible qu'à moitié et on n'a pas accès aux liens d'acceptation ou opposition.
Fichiers
Révisions associées
Historique
Mis à jour par Serghei Mihai (congés, retour 15/05) il y a environ 5 ans
Je suis d'avis d'afficher le bandeau en haut de la page quand on est en mode PWA.
Mis à jour par Serghei Mihai (congés, retour 15/05) il y a environ 5 ans
- Fichier 0001-misc-display-constent-banner-on-page-top-in-pwa-mode.patch 0001-misc-display-constent-banner-on-page-top-in-pwa-mode.patch ajouté
- Fichier pwa-consent-banner.png pwa-consent-banner.png ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
En marquant au préalable dans le corps de la page que c'est une pwa.
Mis à jour par Frédéric Péters il y a environ 5 ans
{% if pwa_display == "standalone" %}pwa-{{ pwa_display }}{% endif %}
Ça aura du mal à produire autre chose que pwa-standalone comme classe.
La navigation est également en bas d'écran quand $nav-mobile-mode == bottom-bar.
C'est curieux qu'il n'y ait pas de media query dans ton patch.
Mis à jour par Serghei Mihai (congés, retour 15/05) il y a presque 5 ans
- Fichier 0001-misc-display-constent-banner-on-page-top-in-pwa-and-.patch 0001-misc-display-constent-banner-on-page-top-in-pwa-and-.patch ajouté
- Assigné à mis à Serghei Mihai (congés, retour 15/05)
Alors juste pwa comme classe sur le body quand il s'agit d'une PWA.
Et prise en compte du mode mobile avec la navigation en bas de la page.
Mis à jour par Frédéric Péters il y a presque 5 ans
- Fichier 0001-misc-display-constent-banner-on-page-top-in-pwa-and-.patch 0001-misc-display-constent-banner-on-page-top-in-pwa-and-.patch ajouté
J'ai repris pour appliquer les modifications suivantes :
- utilisation de 0 le chiffre et pas O la lettre;
- mise en commun du code dans un "sélecteur" %top-consent-banner;
- extension au code pour noircir l'ombre portée et réduire la transparence du fond, sur l'idée que le contenu de l'entête est souvent plus marqué que celui du pied de page;
- augmentation du z-index, ce qui était nécessité au moins pour Quimper.
Mis à jour par Frédéric Péters il y a presque 5 ans
- Fichier 0001-scss-display-consent-banner-at-the-top-when-there-s-.patch 0001-scss-display-consent-banner-at-the-top-when-there-s-.patch ajouté
En corrigeant le message de commit pour tenir dans 80 caractères et en positionnant le bandeau en position: absolute quand il est en haut (plutôt que position: fixed, parce que ça ne sert à rien).
Mis à jour par Serghei Mihai (congés, retour 15/05) il y a presque 5 ans
Frédéric Péters a écrit :
- utilisation de 0 le chiffre et pas O la lettre;
J'ai fait du gros n'importe quoi.
Ack.
Mis à jour par Frédéric Péters il y a presque 5 ans
- Statut changé de Solution proposée à Résolu (à déployer)
+ modif à quelques _vars.scss, pour les intégrations ne chargeant pas _nav.scss.
commit 744f04d23982b58cd48516330705094af5de95c4
Author: Serghei Mihai <smihai@entrouvert.com>
Date: Fri May 10 14:10:58 2019 +0200
scss: display consent banner at the top when there's a bottom bar (#32974)
Mis à jour par Frédéric Péters il y a presque 5 ans
- Statut changé de Résolu (à déployer) à Solution déployée
scss: display consent banner at the top when there's a bottom bar (#32974)