Development #40661
standardiser les "classes réutilisables" sur le préfixe pk-
0%
Description
Au maximum.
Fichiers
Révisions associées
Historique
Mis à jour par Frédéric Péters il y a environ 4 ans
- Fichier 0001-scss-add-pk-prefixed-variants-of-most-reusable-CSS-c.patch 0001-scss-add-pk-prefixed-variants-of-most-reusable-CSS-c.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Le maximum étant ici de ne pas toucher aux classes posées sur la cellule (no-bottom-margin, foldable, folded, links-list), pour différentes raisons : no-bottom-margin parce que ça devrait disparaitre, foldable & folded parce qu'il y a du js associé, links-list parce que c'est massivement utilisé en interne.
Mis à jour par Nicolas Roche il y a environ 4 ans
- Statut changé de Solution proposée à Solution validée
Les classes mobile-only, desktop-only, budget-table et hidden sont dédoublées avec leur homologues toutes préfixées par pk-.
Aucune régression possible à priori.
Mis à jour par Thomas Jund il y a environ 4 ans
Refexions autour de ce ticket.
- Niveau sémantique, je me demande si le prefix `pk-` (pour 'publik') est adéquate pour indiquer les class que l'on peut ajouter côté back combo aux cellules. (Me disant que cette solution de poser des class depuis l'IU de combo est une mauvaise pratique, même au sein de ckEditor).
- Quid des 'class CSS' spécifiquement crééent pour les thèmes, il faudrait aussi un prefix. Serait-il cohérent d'utiliser également le prefix pk- pour uniformiser l'UX côté back ? Ou non car les class custom pourraient entrer en conflit avec le core ?
- Pour inclure `foldable` & `folded` un update du JS ne me parait pas trop compliqué.
En sommes, pour l'utilisateur, utiliser certaines class avec prefix pk, d'autres sans, et certaines avec d'autres prefix, je me demande quelle (in)cohérence on fini par apporter.
Mis à jour par Frédéric Péters il y a environ 4 ans
C'est une progression, ticket par ticket.
Me disant que cette solution de poser des class depuis l'IU de combo est une mauvaise pratique, même au sein de ckEditor).
Et la direction générale reste #39282#note-7.
Mis à jour par Frédéric Péters il y a environ 4 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 074c57367e52c67775d018b37690c5bc8bcae96e
Author: Frédéric Péters <fpeters@entrouvert.com>
Date: Wed Mar 11 17:40:02 2020 +0100
scss: add pk- prefixed variants of most reusable CSS classes (#40661)
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: add pk- prefixed variants of most reusable CSS classes (#40661)