Development #33135
avoir des classes info/warning/etc. notice applicables aux champs w.c.s.
0%
Description
Aujourd'hui ces classes sont définies pour les usages de w.c.s. et sont abusées pour donner du style dans des commentaires, mais ça ne marche pas tout à fait (notamment un conflit au niveau des dimensions quand c'est combiné avec un style grid, cf #33132).
Fichiers
Demandes liées
Révisions associées
Historique
Mis à jour par Frédéric Péters il y a environ 4 ans
- Lié à Development #39282: Mettre à disposition des mises en formes prédéfinies pour des affichages textes ajouté
Mis à jour par Frédéric Péters il y a plus de 3 ans
- Fichier 0001-scss-introduce-pk-information-attention-error-classe.patch 0001-scss-introduce-pk-information-attention-error-classe.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Introduction des styles pk-information, pk-attention, pk-error, qui peuvent s'appliquer sur les champs de type commentaire et sur les cellules et sont documentées et sont à préférer à info/warning/error notice.
J'ai hésité à laisser les styles existants en l'état mais au final je cède et ça améliorera aussi les choses pour l'existant info/warning/error notice.
Mis à jour par Thomas Jund (congés, retour le 29/04) il y a plus de 3 ans
Petites remarques sur le champs d'application de ces nouvelles class.
Je trouve ça assez dangereux au niveau des conflits CSS et des bizarreries graphiques de permettre d'ajouter ces class à toutes les cellules.
Ne serait-il pas plus sage de limiter l'utilisation de ces class sur la cellule texte combo et commentaire wcs uniquement ?
Ensuite, qui dit wysiwyg dit possibilité d'inclure des liens. Ne faudrait-il pas les styler également ?
Mis à jour par Frédéric Péters il y a plus de 3 ans
J'étais sur le point d'être d'accord puis non il y aura au moins des cellules créées via JSON où ça pourra être souhaité, je préfère du coup ne pas poser de limitation ici et expliquer à la personne qui mettra ça sur une cellule de code de suivi que ça n'est pas vraiment fait pour ça.
Ensuite, qui dit wysiwyg dit possibilité d'inclure des liens. Ne faudrait-il pas les styler également ?
Il y a déjà des liens posés dans l'existant (visibles dans les captures de #33132, soulignés par défaut depuis), je préfère ne pas mêler ici un changement de style.
Mis à jour par Thomas Jund (congés, retour le 29/04) il y a plus de 3 ans
Testé sur une text-cell avec la class pk-error.
- Par défaut, le titre d'une cellule a un background blanc, il faudrait forcer un background transparent.
- J'ai de gros doute sur le contraste entre couleur de la cellule et couleur du titre de la cellule (color: darken($color, 40%) !important;. Sur une cellule en pk-error, le contraste est insuffisant. Moi j'aurais forcé les titres en $font-color.
- padding-left: $notification-left-padding; est appliqué 2x: 1x sur .cell et une seconde fois sur .cell > div.
Mis à jour par Frédéric Péters il y a plus de 3 ans
Par défaut, le titre d'une cellule a un background blanc, il faudrait forcer un background transparent.
De fait je testais avec une intégration où c'était déjà le cas.
J'ai de gros doute sur le contraste entre couleur de la cellule et couleur du titre de la cellule (color: darken($color, 40%) !important;. Sur une cellule en pk-error, le contraste est insuffisant. Moi j'aurais forcé les titres en $font-color.
Firefox me dit 3.70 validé AA; pour accentuer l'affaire j'y ai forcé le gras.
padding-left: $notification-left-padding; est appliqué 2x: 1x sur .cell et une seconde fois sur .cell > div.
De fait.
Branche à jour avec ces changements.
Mis à jour par Thomas Jund (congés, retour le 29/04) il y a plus de 3 ans
- Statut changé de Solution proposée à Solution validée
cell + class de notif + .grid-* ne fonctionne pas.
Problème de box-sizing.
Je propose de passer les notifs en border-box par défaut.
J'ai poussé un patch dans la branch pour cela.
OK pour le reste.
Mis à jour par Frédéric Péters il y a plus de 3 ans
- Statut changé de Solution validée à En cours
cell + class de notif + .grid-* ne fonctionne pas.
Tu peux expliciter comment ça ne fonctionne pas ?
Je propose de passer les notifs en border-box par défaut.
Je ne mesure pas ces conséquences.
Mis à jour par Thomas Jund (congés, retour le 29/04) il y a plus de 3 ans
Les notifs sont en `content-box`.
La largeur des cell.grid sont définis avec `calc( 50% - 1rem);`
Et comme les notifs ont un padding de .5em => largeur totale non respectée de (50% - 1rem + 1rem).
Il faut 50% padding compris.
Soit on surcharge en `border-box` comme tu as eu besoin de le faire pour les .comment-field.grid_*.
Soit on bascule directement le composant en border-box, comme ça inutile de surcharger (ma proposition).
Je ne mesure pas ces conséquences.
Je n'en vois aucune.
Mis à jour par Frédéric Péters il y a plus de 3 ans
- Fichier 0001-scss-introduce-pk-information-attention-error-classe.patch 0001-scss-introduce-pk-information-attention-error-classe.patch ajouté
- Statut changé de En cours à Solution proposée
Mis à jour par Thomas Jund (congés, retour le 29/04) il y a plus de 3 ans
- Statut changé de Solution proposée à Solution validée
Mis à jour par Frédéric Péters il y a plus de 3 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 9856b660866588418c75303be5d9b70f17411096
Author: Frédéric Péters <fpeters@entrouvert.com>
Date: Wed Jun 24 18:56:15 2020 +0200
scss: introduce pk-information/attention/error classes (#33135)
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
Mis à jour par Brice Mallet il y a plus de 3 ans
- Lié à Bug #47625: avoir une classe pk-success applicable aux champs w.c.s. ajouté
scss: introduce pk-information/attention/error classes (#33135)