Projet

Général

Profil

Development #29227

classe CSS pour différencier le bloc de connexion du bloc de "inscription après login sur FC"

Ajouté par Frédéric Péters il y a plus de 5 ans. Mis à jour il y a plus de 4 ans.

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

0%

Temps estimé:
Hors marché:
Non
Patch proposed:
Oui
Planning:
Non

Description

Pour le moment les blocs ne peuvent pas être différenciés, ce qui bloque certaines réalisations graphiques.


Fichiers


Demandes liées

Lié à Intégrations graphiques Publik - Development #35949: prise en compte des classes CSS pour chaque type d'authentificationFermé09 septembre 2019

Actions

Révisions associées

Révision 2884230f (diff)
Ajouté par Benjamin Dauvergne il y a plus de 4 ans

auth_fc: differentiate registration login block with a class (#29227)

Historique

#2

Mis à jour par Benjamin Dauvergne il y a plus de 4 ans

C'est dans authentic2_auth_fc/{login.html,login_registration.html} je veux bien mettre tout le monde dans button-wrapper avec une classe différenciante, comme cela :

diff --git a/src/authentic2_auth_fc/templates/authentic2_auth_fc/login.html b/src/authentic2_auth_fc/templates/authentic2_auth_fc/login.html
index c88d4f69..9633c34f 100644
--- a/src/authentic2_auth_fc/templates/authentic2_auth_fc/login.html
+++ b/src/authentic2_auth_fc/templates/authentic2_auth_fc/login.html
@@ -2,7 +2,7 @@
 {% load i18n %}

 <link rel="stylesheet" type="text/css" href="{% static 'authentic2_auth_fc/css/fc.css' %}">
-<div id="fc-button-wrapper">
+<div id="fc-button-wrapper fc-login">
   <div id="fc-button">
     <a href="{{ login_url }}" 
        class="button connexion{% if popup %} js-fc-popup{% endif %}">
diff --git a/src/authentic2_auth_fc/templates/authentic2_auth_fc/login_registration.html b/src/authentic2_auth_fc/templates/authentic2_auth_fc/login_registration.html
index 4916a1ec..258cc65a 100644
--- a/src/authentic2_auth_fc/templates/authentic2_auth_fc/login_registration.html
+++ b/src/authentic2_auth_fc/templates/authentic2_auth_fc/login_registration.html
@@ -3,15 +3,14 @@

 <link rel="stylesheet" type="text/css" href="{% static 'authentic2_auth_fc/css/fc.css' %}">

-<div id="fc-registration-user-info">
- <h3>{% trans "Create your account with FranceConnect" %}</h3>
- <p class="certified">
-  <span class="fc-user-full-name">{{ fc_user_info.given_name }} {{ fc_user_info.family_name }}</span>
-  {% if fc_user_info.email %}<span class="fc-user-email">{{ fc_user_info.email }}</span>{% endif %}
- </p>
-</div>
-
-<div id="fc-button-wrapper">
+<div id="fc-button-wrapper fc-registration">
+  <div id="fc-registration-user-info">
+   <h3>{% trans "Create your account with FranceConnect" %}</h3>
+   <p class="certified">
+    <span class="fc-user-full-name">{{ fc_user_info.given_name }} {{ fc_user_info.family_name }}</span>
+    {% if fc_user_info.email %}<span class="fc-user-email">{{ fc_user_info.email }}</span>{% endif %}
+   </p>
+  </div>
   <div id="fc-button">
     <a href="{{ registration_url }}" 
        class="button connexion{% if popup %} js-fc-popup{% endif %}">

Ça irait ?

#3

Mis à jour par Frédéric Péters il y a plus de 4 ans

tout le monde dans button-wrapper
(...)
Ça irait ?

Pas bien sûr des conséquences, il y a pas mal de style déjà appliqué à div#fc-button-wrapper, du coup je pense plus sûr que fc-button-wrapper reste uniquement à entourer fc-button.

À l'origine de ce ticket c'était l'intégration graphique Quimper mais je ne me souviens plus du point particulier que j'aurais aimé y traiter; par contre, pour le point récent qui serait de pouvoir faire monter le bloc FC au-dessus du bloc de connexion par mot de passe, au retour de FC, en mobile, ça doit se jouer plus haut, en gros, pouvoir arriver à quelque chose comme :

 {% block login-blocks %}
 {% for id, login_block in blocks.iteritems %}
-<div class="block block-{{id}}">
+<div class="block block-{{id}} {{ login_block.extra_css_class }}">
#4

Mis à jour par Benjamin Dauvergne il y a plus de 4 ans

  • Assigné à mis à Benjamin Dauvergne
#5

Mis à jour par Benjamin Dauvergne il y a plus de 4 ans

#6

Mis à jour par Serghei Mihai il y a plus de 4 ans

Tu peux mettre à jour le patch par rapport à master?
Aussi, dans authentic2_auth_fc/authenticators.py tu peux remplacer le render(...) qui reste par TemplateResponse et virer l'import ?

from django.shortcuts import render

#8

Mis à jour par Serghei Mihai il y a plus de 4 ans

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

Go.
Il faut rajouter la déclaration de la classe dans publik-base-theme (j'ai pas trouvé de ticket).

#9

Mis à jour par Serghei Mihai il y a plus de 4 ans

  • Lié à Development #35949: prise en compte des classes CSS pour chaque type d'authentification ajouté
#10

Mis à jour par Benjamin Dauvergne il y a plus de 4 ans

  • Statut changé de Solution validée à Résolu (à déployer)
commit 2884230fe48148805ef30ad6c82d3723fb8de3ba
Author: Benjamin Dauvergne <bdauvergne@entrouvert.com>
Date:   Sat Aug 10 15:12:43 2019 +0200

    auth_fc: differentiate registration login block with a class (#29227)
#11

Mis à jour par Frédéric Péters il y a plus de 4 ans

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

Formats disponibles : Atom PDF