Projet

Général

Profil

Development #46875

Pouvoir utiliser les class .pk-table-* avec @extend

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:
22 septembre 2020
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

Les templates solis apa utilisent les class .pk-table-* pour mettre en forme les tableaux.
Mauvaise pratique (à mon avis) de fixer les styles de ces éléments au sein des templates HTML.

Il serait préférable d'appliquer ces styles via custom.scss du thème.

.rsa-cell table {
  @extend .pk-data-table;
  @extend .pk-table-headers;
  @extend .pk-table-borders;
  @extend .pk-table-zebra;
}

Il faut pour cela exposer ces class CSS de manière à les rendre disponible à la fonction @extend.


Fichiers


Demandes liées

Lié à Intégrations graphiques Publik - Development #46464: ajout des templates Solis RSAFermé23 septembre 2020

Actions

Révisions associées

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

css: refactor pk-table-* selectors to be usable with @extend sass function (#46875)

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

Mauvaise pratique (à mon avis) de fixer les styles de ces éléments au sein des templates HTML.

Le déroulé c'est que ces classes sont là pour permettre de styler lors du prototypage, où le CPF est indépendant, où il n'y a pas de modification des intégrations.

Elles sont là pour ça, elles sont garanties maintenues, je ne vois pas vraiment le gain à ne plus les utiliser lors de l'intégration du code des cellules.

#3

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

Si utilisé dans un template au sein d'un theme, ou en prototypage, oui je ne vois aucun inconvénient.
Dans ce cas, l'objectif est de pousser les templates RSA dans le core, pour être exploité par d'autres thèmes.

#4

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

Oui mais je ne vois pas ce que ça change, les classes .pk-* doivent de toute façon être maintenues et fonctionner partout.

#5

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

#6

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

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

Plutôt to be usable dans le message de commit.

#7

Mis à jour par Thomas Noël il y a plus de 3 ans

  • Statut changé de Solution validée à Résolu (à déployer)
commit 1488aeb2f52e83e74f7463b067183bd8bc8ef15e
Author: Thomas JUND <tjund@entrouvert.com>
Date:   Tue Sep 22 11:41:14 2020 +0200

    css: refactor pk-table-* selectors to be usable with @extend sass function (#46875)

#8

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

Formats disponibles : Atom PDF