Produits Entr'ouvert » Intégrations graphiques Publik

Moi je ne comprends pas, pourquoi le markup d'une links-list est utilisé alors que l'option "tableau" est demandé.
Même pour 1 seul item
C'est peut-être ça qu'il faut questionner aussi.

Et puis toujours dans ce cas bizarre d'une seul item en mode tableau qui retourne un links-list. Le markup est inséré au sein de la li.
Donc si on choisi d'afficher un champ de la fiche quelconque, on va avoir une links-list sans link, sans intérêt aucun et surtout un truc impossible à gérer côté CSS

<li>
                delta
              </li>

</ul>
        </div>

Il faut un tableau.

Moi je ne comprends pas, pourquoi le markup d'une links-list est utilisé alors que l'option "tableau" est demandé.

Il est noté où que l'option tableau est demandée ? Si ça arrive et qu'il n'y a pas de balisage tableau utilisé, il faut faire un ticket combo. Ça n'enlève rien au fait qu'un bouton préesnt dans un affichage choisi sous forme de liste doivent s'afficher correctement (ce ticket).

Ça n'enlève rien au fait qu'un bouton préesnt dans un affichage choisi sous forme de liste doivent s'afficher correctement (ce ticket).

Mais à préciser.

Je ne sais pas si un `a.pk-button` au sein d'une liste de liens a des raisons d'exister (ce dont parle ce ticket). Je parle de `.links-list ul li a.pk-button` : pouvoir différencier un lien sous forme de bouton et laisser les autres sous forme de liens ?

Par contre une class posée sur une cellule comme `cell.pk-button .links-list ul li a` ou du `cell.pk-buttons-list .links-list ul li a` oui doit être traité, (tous les liens s'affichent sous forme de boutons) mais ce n'est pas le même code ni les mêmes contraintes.

Alors j'ai fait le tour, c'est un choix de compatibilité pour une liste de fiches configurée pour afficher une seule info (lire "colonne") de reprendre le balisage de liste. (plutôt que faire un tableau avec une seule colonne).

Cela posé, donc, on a cette option de mode d'affichage "bouton" pour les liens, et ça peut donc être appliqué sur le lien unique qui va apparaître dans une liste, bref ce ticket.

Et le sujet serait que l'affichage soit correct, et le patch proposé par Corentin assure ça.

Moi ça m'irait qu'il soit validé, que le problème de contraste actuellement réel puisse ainsi être résolu, et que s'il y a des trucs plus profonds sur lesquels réfléchir de nouveaux tickets soient créés.

Alors j'ai fait le tour, c'est un choix de compatibilité pour une liste de fiches configurée pour afficher une seule info (lire "colonne") de reprendre le balisage de liste

Ok, pourquoi pas afficher une liste plutôt qu'un tableau dans le cas 1 seule "colonne", mais pourquoi une class .links-list ?
Idée que je ne trouve pas du tout maitrisés et assez mauvaise, si c'est juste pour récupérer les bordures de séparation.
La class links-list embarque des styles dans l'objectif est de lister une liste de liens et n'est pas prévue pour autre chose.

et que s'il y a des trucs plus profonds sur lesquels réfléchir de nouveaux tickets soient créés.

Comme indiqué plus haut (dans le message super mal formaté), on peut formater le contenu d'un item de fiche de plein de manière et cela va certainement continuer en ajoutant de nouvelles options. Avoir `links-list > ul > li > `machin` autre que `a` n'est pas géré au niveau CSS et l'affichage est potentiellement mauvais. Le problème ne se limite surement pas aux pk-button.
Alors plutôt que de vouloir faire de .links-list un truc fourre tout qui fait plein de chose autre que des listes de liens, juste pour le cas unique d'un affichage tableau avec 1 seul item, je serais pour cloisoner un peu les choses et éviter les conflits.

Moi ça m'irait qu'il soit validé

ok, mais en contreparti supprimons cette links-list.

ok, mais en contreparti supprimons cette links-list.

On est d'accord que ça veut dire soit 1/ ajouter une couche de compatibilité dans publik-base-theme pour que l'affichage actuel de fiches affichées comme liste de liens (pas de boutons) continue à fonctionner; soit 2/ détecter la situation particulière de "tableau une seule colonnne et dedans c'est bien un lien configuré pour être affiché sous forme de lien" pour continuer à y mettre links-list. (tu peux créer un ou deux tickets selon tes préférences).

Corentin Sechet (csechet) a fermé une pull request sur Gitea concernant cette demande.