Projet

Général

Profil

Development #46136

Introduire variable pour affichage des "OU"

Ajouté par Thomas Jund il y a plus de 3 ans. Mis à jour il y a plus de 3 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
27 août 2020
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

le séparateur "OU" entre les blocks d'authentification et conditionné par la présence ou non d'une bordure aux cellules.

Je trouve cette condition trop légère.
Dans bien d'autres cas, les "ou" ne sont pas propices:

  • Si les blocks ont un design différent des cellules
  • Si les blocks ont un background, ou un box-shadow
  • etc.

Le hack "$cell-border = 0" est peu lisible.
Je propose d'introduire un booleen pour conditionner son affichage en plus de conserver la condition sur la présence du border (qui facilite l'intégration dans beaucoup de cas).


Fichiers


Demandes liées

Lié à Intégrations graphiques Publik - Bug #46291: Correction $or-separatorFermé02 septembre 2020

Actions

Révisions associées

Révision 0110f86d (diff)
Ajouté par Thomas Jund il y a plus de 3 ans

scss: introduce $or-separator for authentication blocks (#46136)

Révision 366d677b (diff)
Ajouté par Thomas Jund il y a plus de 3 ans

remove git conflict marker (#46136)

rel commit: 0110f86d

Historique

#1

Mis à jour par Thomas Jund il y a plus de 3 ans

#2

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

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

Mis à jour par Thomas Jund il y a plus de 3 ans

  • Statut changé de Solution validée à Résolu (à déployer)
commit 0110f86d1859c9e6a3a1b3b05e6701751b6e8bcc (HEAD -> master, origin/master, origin/HEAD)
Author: Thomas JUND <tjund@entrouvert.com>
Date:   Thu Aug 27 12:36:36 2020 +0200

    scss: introduce $or-separator for authentication blocks (#46136)
#4

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

  • Statut changé de Résolu (à déployer) à En cours

Ça été poussé avec des marqueurs de conflit.

+<<<<<<< HEAD
 $notification-left-padding: 4rem;
 @if $notification-style == border-icon {
     $notification-left-padding: 2rem;
+=======
+// add a border with "OU" between authentication blocks
#7

Mis à jour par Thomas Jund il y a plus de 3 ans

#8

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

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

Mis à jour par Thomas Jund il y a plus de 3 ans

  • Statut changé de Solution validée à Résolu (à déployer)
commit 366d677b30ae45d635323caf78d7ffe97d1f7941 (HEAD -> master, origin/master, origin/HEAD)
Author: Thomas JUND <tjund@entrouvert.com>
Date:   Fri Aug 28 15:47:37 2020 +0200

    remove git conflict marker (#46136)

    rel commit:  0110f86d
#10

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

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

Mis à jour par Thomas Jund il y a plus de 3 ans

  • Lié à Bug #46291: Correction $or-separator ajouté

Formats disponibles : Atom PDF