Development #57570
Cellule recherche: personalisation bouton côté thème
0%
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
Historique
Mis à jour par Thomas Jund il y a plus de 2 ans
- Fichier 0001-search-cell-improve-markup-to-target-button-and-butt.patch 0001-search-cell-improve-markup-to-target-button-and-butt.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Serghei Mihai il y a plus de 2 ans
- Statut changé de Solution proposée à Solution validée
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é
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.
Mis à jour par Thomas Jund il y a plus de 2 ans
- Fichier 0001-search-cell-improve-markup-to-target-button-and-butt.patch 0001-search-cell-improve-markup-to-target-button-and-butt.patch ajouté
- Statut changé de En cours à Solution proposée
Avec {% block submit-label %}
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 ?
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 %}
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.
Mis à jour par Thomas Jund il y a plus de 2 ans
- Fichier 0001-search-cell-improve-markup-to-target-button-and-butt.patch 0001-search-cell-improve-markup-to-target-button-and-butt.patch ajouté
{% block submit-content %} inside <span>
Mis à jour par Frédéric Péters il y a plus de 2 ans
- Statut changé de Solution proposée à Solution validée
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)
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
search-cell: improve markup to target button and button label (#57570)