Projet

Général

Profil

Development #57570

Cellule recherche: personalisation bouton côté thème

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

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
04 octobre 2021
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

Pour permettre d'utiliser les icons fontawesome à la place du label (comme icone loupe) uniquement coté theme sans modifier le design côté portail agent ni surcharger le template il faudrait ajouter un balise autour du label.


Fichiers

Révisions associées

Révision 377cbcc9 (diff)
Ajouté par Thomas Jund il y a plus de 2 ans

search-cell: improve markup to target button and button label (#57570)

Historique

#1

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

#2

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

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

Mis à jour par Pierre Cros il y a plus de 2 ans

Je suis preneur d'un lien vers un exemple quand ce sera déployé

#4

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

  • Statut changé de Solution validée à En cours

@Pierre : Je ne sais pas si c'était clair, il va juste y avoir dans le code <button><span>Rechercher</span></button> plutôt que <button>Rechercher</button>, c'est juste une modification de balisage pour permettre d'ajouter certaines CSS (certaines fonctionnalités ne fonctionnent pas sur <button>).

~~

Sur le patch en lui-même je préférerais vraiment que le {% block submit-content %} contienne uniquement le libellé, qu'une intégration qui demanderait un libellé différent n'ait pas à recopier <span class="combo-search--button-label"> pour garder le même balisage.

#5

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

Avec {% block submit-label %}

#6

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

À quoi ça sert de doubler ainsi ? Quelle situation où on voudra ne pas avoir <span class="combo-search--button-label">, qui est à mon sens du balisage totalement inerte ?

#7

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

Par forcemeent "ne pas avoir", je pensais plutôt ajouter à coté ou autour

{% block button-content %}
 whatever (img, svg, text, icon) 
 {{ block.super }}
{% endblock %}
#8

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

Contenu additionnel qui pourrait très bien aller dans le <span class="combo-search--button-label">, pas besoin de pouvoir le retirer.

J'insiste ici parce que multiplier les points d'extension c'est complexifier les évolutions.

#10

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

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

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

  • Statut changé de Solution validée à Résolu (à déployer)
commit 377cbcc95560c5312aa39574c02d843d722f42d9
Author: Thomas JUND <tjund@entrouvert.com>
Date:   Mon Oct 4 17:35:36 2021 +0200

    search-cell: improve markup to target button and button label (#57570)
#12

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

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

Formats disponibles : Atom PDF