Development #40660
pouvoir appliquer pk-(big-)button à une cellule
0%
Description
Ce sont actuellement des classes qui s'appliquent à des liens dans du texte, il y a demande à pouvoir également les appliquer sur des cellules de type lien.
Fichiers
Révisions associées
Historique
Mis à jour par Frédéric Péters il y a environ 4 ans
- Fichier 0001-scss-make-it-possible-to-apply-pk-big-button-onto-li.patch 0001-scss-make-it-possible-to-apply-pk-big-button-onto-li.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Nicolas Roche il y a environ 4 ans
- Statut changé de Solution proposée à Solution validée
La classe s'applique bien aux cellules de type lien (peut-être préciser le titre du ticket).
Mis à jour par Thomas Jund il y a environ 4 ans
Je comprends que par facilité on ai envie de faire ça, mais pour moi, c'est une mauvaise idée.
La class "pk-button" a pour objectif de donner le design d'un bouton à un element de type <a> <button> ou <submit> directement sur l'élément et qui sont les éléments nativement cliquables.
Son rôle est encore claire. Je ne suis pas pour ajouter de la complexité et des exceptions.
Soit on créé une nouvelle class qui indique : "transforme la cell link en bouton". Un modificateur à la cellule type lien.
Ou créer une class plus générique qui a pour rôle "transforme tous les liens enfants en bouton"
- .link-cell-as-pk-button
- .pk-button-inside
- .like-pk-button (bof)
Soit on fait en sorte que la cell link retourne vraiment un lien et pas une `links-list` d'un seul item (qui est sémantiquement pas terrible).
Mis à jour par Frédéric Péters il y a environ 4 ans
Son rôle est encore claire. Je ne suis pas pour ajouter de la complexité et des exceptions.
Je trouve claire la proposition ici, et logique du point de vue de l'usager qu'un lien, que ça soit dans du texte ou par une cellule "lien", puisse être qualifié de la classe. (comme on a ailleurs des demandes pour que des classes "tableau" puissent s'appliquer aux champs tableaux de w.c.S.).
Mis à jour par Thomas Jund il y a environ 4 ans
Je trouve claire la proposition ici, et logique du point de vue de l'usager qu'un lien
Entièrement d'accord si point de vue de l'usager. Mon point de vue est plus coté structure / complexité du code. "Single responsability".
Mis à jour par Frédéric Péters il y a environ 4 ans
- Statut changé de Solution validée à Résolu (à déployer)
Soit on fait en sorte que la cell link retourne vraiment un lien et pas une `links-list` d'un seul item (qui est sémantiquement pas terrible).
Oui c'est quelque chose qui pourra être fait, maintenant qu'on a une cellule "liste de liens"; mais il faudra d'abord passer sur l'existant pour le convertir.
commit 14ed15d7f80bc85c1ee540e7dae4f62f2a44ebfa
Author: Frédéric Péters <fpeters@entrouvert.com>
Date: Wed Mar 11 17:33:50 2020 +0100
scss: make it possible to apply pk-(big-)button onto link cells (#40660)
Mis à jour par Frédéric Péters il y a environ 4 ans
- Statut changé de Résolu (à déployer) à Solution déployée
scss: make it possible to apply pk-(big-)button onto link cells (#40660)