Projet

Général

Profil

Development #72519

toodego: dans le template authentic2/authentic2/display_message_and_continue.html le lien ne devrait pas être caché

Ajouté par Benjamin Dauvergne il y a plus d'un an. Mis à jour il y a environ un an.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
15 décembre 2022
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

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

Lié à Authentic 2 - Development #33703: arrêt pour afficher un message "vous avez été déconnecté"Fermé05 juin 2019

Actions

Révisions associées

Révision 3c890972 (diff)
Ajouté par Benjamin Dauvergne il y a environ un an

misc: remove useless overloading of authentic2/display_message_and_continue.html (#72519)

Historique

#1

Mis à jour par Benjamin Dauvergne il y a plus d'un an

  • Assigné à mis à Benjamin Dauvergne
#2

Mis à jour par Benjamin Dauvergne il y a plus d'un an

#4

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.

#6

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 ?

#9

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é
#10

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.

#12

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 () {
#13

Mis à jour par Benjamin Dauvergne il y a environ un an

En retirant les deux.

#14

Mis à jour par Robot Gitea il y a environ un an

Benjamin Dauvergne (bdauvergne) a ouvert une pull request sur Gitea concernant cette demande :

#16

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 :

#17

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 :

#18

Mis à jour par Transition automatique il y a environ un an

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

Mis à jour par Transition automatique il y a 12 mois

Automatic expiration

Formats disponibles : Atom PDF