Development #72519
toodego: dans le template authentic2/authentic2/display_message_and_continue.html le lien ne devrait pas être caché
0%
Description
Si les messages affichés contiennent des erreurs, le click automatique sur le lien "Continuer" n'a pas lieu (only_info==True), mais comme le lien est caché, l'utilisateur se retrouve bloqué.
{% extends "authentic2/base.html" %} {% load i18n %} {% block content %} <div class="a2-continue" id="a2-continue-container" style="display: none"> <a id="a2-continue" href="{{ url }}" style="display: none">{% trans "Continue" %}</a> </div> {% if only_info %} <script> if (document.getElementById('messages').innerHTML.indexOf("Vous avez été déconnecté") != -1) { document.getElementById('a2-continue').click(); } else { document.getElementById('a2-continue-container').style.display = 'block'; window.setTimeout(function () { document.getElementById('a2-continue').click(); }, 3000); } </script> {% endif %} {% endblock %}
Fichiers
Demandes liées
Révisions associées
Historique
Mis à jour par Benjamin Dauvergne il y a plus d'un an
- Fichier 0001-toodego-show-continue-link-if-messages-contains-warn.patch 0001-toodego-show-continue-link-if-messages-contains-warn.patch ajouté
- Tracker changé de Support à Development
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Emmanuel Cazenave il y a plus d'un an
Le patch me semble cacher le lien s'il y a des erreurs alors que tu veux l'inverse si j'en crois le message de commit.
Mis à jour par Benjamin Dauvergne il y a plus d'un an
- Fichier 0001-toodego-show-continue-link-if-messages-contains-warn.patch 0001-toodego-show-continue-link-if-messages-contains-warn.patch ajouté
Fatigue.
Mis à jour par Paul Marillonnet il y a plus d'un an
Et du coup peut-être retirer le style="display: none" du lien contenu dans cette div ?
Mis à jour par Benjamin Dauvergne il y a plus d'un an
- Fichier 0001-toodego-show-continue-link-if-messages-contains-warn.patch 0001-toodego-show-continue-link-if-messages-contains-warn.patch ajouté
Bien vu. Il est formidable ce template.
Mis à jour par Benjamin Dauvergne il y a plus d'un an
- Lié à Development #33703: arrêt pour afficher un message "vous avez été déconnecté" ajouté
Mis à jour par Benjamin Dauvergne il y a plus d'un an
En fait depuis #33703 je pense que ce template n'est même plus nécessaire pour l'objectif qu'il visait.
Mis à jour par Benjamin Dauvergne il y a environ un an
Mis à jour par Benjamin Dauvergne il y a environ un an
- Statut changé de Solution proposée à Résolu (à déployer)
Le dernier patch retire complètement ce template pour utiliser la version standard de pbt (qui ne me semble pas plus utile, je pourrai vraisemblablement retirer les deux) :
commit 0f03229c6c1c8f05180a9287e9d724b2a920c73f (tag: v1.25) Author: Frédéric Péters <fpeters@entrouvert.com> Date: Wed May 3 23:15:29 2017 +0200 misc: add proper styling to "message and continue" authentic page (#16163) diff --git a/templates/authentic2/display_message_and_continue.html b/templates/authentic2/display_message_and_continue.html new file mode 100644 index 00000000..7c13d287 --- /dev/null +++ b/templates/authentic2/display_message_and_continue.html @@ -0,0 +1,15 @@ +{% extends "authentic2/base.html" %} +{% load i18n %} + +{% block content %} + <div class="a2-continue"> + <a id="a2-continue" href="{{ url }}">{% trans "Continue" %}</a> + </div> + {% if only_info %} + <script> + window.setTimeout(function () { + document.getElementById('a2-continue').click(); + }, 3000); + </script> + {% endif %} +{% endblock %}
et version actuellement dans a2:
commit 35bf68f922c6abc1654ea31fc60bdc0166eca4f0 (tag: v3.70) Author: Benjamin Dauvergne <bdauvergne@entrouvert.com> Date: Wed Jan 19 13:13:52 2022 +0100 misc: prevent double click in display_message_and_continue.html (#60815) diff --git a/src/authentic2/templates/authentic2/display_message_and_continue.html b/src/authentic2/templates/authentic2/display_message_and_continue.html index 2f378442..958e719f 100644 --- a/src/authentic2/templates/authentic2/display_message_and_continue.html +++ b/src/authentic2/templates/authentic2/display_message_and_continue.html @@ -7,9 +7,21 @@ </div> {% if only_info %} <script> - window.setTimeout(function () { - document.getElementById('a2-continue').click(); - }, 3000); + $(function () { + var $a2_continue = $('#a2-continue'); + var clicked = false; + $a2_continue.on('click', function (event) { + if (! clicked) { + # prevent double click for 3 seconds + clicked = true; + window.setTimeout(function () { clicked = false; }, 3000); + } else { + event.preventDefault(); + } + }); + # automatic click after 3 seconds + window.setTimeout(function () { if (!clicked) { $a2_continue.click(); } }, 3000); + }); </script> {% endif %} {% endblock %} commit 7dce0d930d3d720bf9d82cdaab544bf425ae4368 Author: Frédéric Péters <fpeters@entrouvert.com> Date: Wed May 3 17:38:07 2017 +0200 misc: include "continue" link in a <div> (#16159) diff --git a/src/authentic2/templates/authentic2/display_message_and_continue.html b/src/authentic2/templates/authentic2/display_message_and_continue.html index fa324a09..2f378442 100644 --- a/src/authentic2/templates/authentic2/display_message_and_continue.html +++ b/src/authentic2/templates/authentic2/display_message_and_continue.html @@ -2,7 +2,9 @@ {% load i18n %} {% block content %} - <a id="a2-continue" href="{{ url }}">{% trans "Continue" %}</a> + <div class="a2-continue"> + <a id="a2-continue" href="{{ url }}">{% trans "Continue" %}</a> + </div> {% if only_info %} <script> window.setTimeout(function () {
Mis à jour par Benjamin Dauvergne il y a environ un an
- Fichier 0001-misc-remove-useless-overloading-of-authentic2-displa.patch 0001-misc-remove-useless-overloading-of-authentic2-displa.patch ajouté
- Statut changé de Résolu (à déployer) à Solution proposée
En retirant les deux.
Mis à jour par Robot Gitea il y a environ un an
Benjamin Dauvergne (bdauvergne) a ouvert une pull request sur Gitea concernant cette demande :
- URL : https://gitea.entrouvert.org/entrouvert/publik-base-theme/pulls/115
- Titre : misc: remove useless overloading of authentic2/display_message_and_continue.html (#72519)
- Modifications : https://gitea.entrouvert.org/entrouvert/publik-base-theme/pulls/115/files
Mis à jour par Robot Gitea il y a environ un an
- Statut changé de Solution proposée à Solution validée
Paul Marillonnet (pmarillonnet) a approuvé une pull request sur Gitea concernant cette demande :
Mis à jour par Robot Gitea il y a environ un an
- Statut changé de Solution validée à Résolu (à déployer)
Frédéric Péters (fpeters) a mergé une pull request sur Gitea concernant cette demande :
- URL : https://gitea.entrouvert.org/entrouvert/publik-base-theme/pulls/115
- Titre : misc: remove useless overloading of authentic2/display_message_and_continue.html (#72519)
- Modifications : https://gitea.entrouvert.org/entrouvert/publik-base-theme/pulls/115/files
Mis à jour par Transition automatique il y a environ un an
- Statut changé de Résolu (à déployer) à Solution déployée
misc: remove useless overloading of authentic2/display_message_and_continue.html (#72519)