Bug #19496
ajouter les classes "foldable" et "folded"
100%
Description
Ces permettront de désigner un élément comme "pliable" ou "dépliable".
Fichiers
Demandes liées
Révisions associées
Historique
Mis à jour par Josué Kouka il y a plus de 6 ans
- Lié à Development #19467: Avoir des classes CSS standard (et du js de support) pour gérer des cellules repliées/repliables ajouté
Mis à jour par Josué Kouka il y a plus de 6 ans
- Fichier 0001-cells-add-class-for-folded-and-foldable-cells-19496.patch 0001-cells-add-class-for-folded-and-foldable-cells-19496.patch ajouté
- Patch proposed changé de Non à Oui
Mis à jour par Frédéric Péters il y a plus de 6 ans
Là ça va chopper tous les h2 des cellules. faudrait juste attraper celui qui est le titre de la cellule #content div.cell h2:first-child.
Déjà répété mille fois, quand on fait référence à un caractère perdu au milieu de FontAwesome, on précise en commentaire à côté le caractère en question, ex: content: '\f05a'; /* info-circle */.
Mis à jour par Josué Kouka il y a plus de 6 ans
- Fichier 0001-cells-add-class-for-folded-and-foldable-cells-19496.patch 0001-cells-add-class-for-folded-and-foldable-cells-19496.patch ajouté
Frédéric Péters a écrit :
Là ça va chopper tous les h2 des cellules. faudrait juste attraper celui qui est le titre de la cellule
#content div.cell h2:first-child.Déjà répété mille fois, quand on fait référence à un caractère perdu au milieu de FontAwesome, on précise en commentaire à côté le caractère en question, ex:
content: '\f05a'; /* info-circle */.
Ok corrigé.
Mis à jour par Frédéric Péters il y a plus de 6 ans
Sans énormes explications, ne jamais utiliser !important.
Mis à jour par Josué Kouka il y a plus de 6 ans
Frédéric Péters a écrit :
Sans énormes explications, ne jamais utiliser !important.
Désolé. J'ai utilisé !important parce que quand un element a à la fois les classes foldable et folded appliqués, c'est le style lié à la classe foldable qui prévaut.
Mis à jour par Frédéric Péters il y a plus de 6 ans
Pas valable.
Bêtement déclarer foldable avant folded. Ou vu que folded marche uniquement accompagné de foldable et utiliser les deux classes (.foldable.folded).
Mis à jour par Josué Kouka il y a plus de 6 ans
- Fichier 0001-cells-add-class-for-folded-and-foldable-cells-19496.patch 0001-cells-add-class-for-folded-and-foldable-cells-19496.patch ajouté
Frédéric Péters a écrit :
Pas valable.
Bêtement déclarer foldable avant folded. Ou vu que folded marche uniquement accompagné de foldable et utiliser les deux classes (.foldable.folded).
Ok, pris en compte. Le style pour plier le contenu de la cellule s'applique lorsque les classes foldable et folded sont presentes.
Mis à jour par Frédéric Péters il y a plus de 6 ans
Ce fichier est indenté avec des tabulations. La logique voudrait .foldable puis .foldable.folded. Documenter ces classes dans help/fr/misc-css-classes.page.
Mis à jour par Josué Kouka il y a plus de 6 ans
- Fichier 0001-cells-add-class-for-folded-and-foldable-cells-19496.patch 0001-cells-add-class-for-folded-and-foldable-cells-19496.patch ajouté
Frédéric Péters a écrit :
Ce fichier est indenté avec des tabulations. La logique voudrait .foldable puis .foldable.folded. Documenter ces classes dans help/fr/misc-css-classes.page.
Documentation mise a jour, espaces remplaces par des tabs et le foldable avant le .folfable.folded.
Mis à jour par Frédéric Péters il y a plus de 6 ans
Les autres descriptions se terminent pas des points. "le contenu d'un cellule " → d'unE. AssociéE vu qu'on parle d'unE classe. "cette classe définit" : pour suivre les autres points on écrirait plutôt simplement "définit".
Mis à jour par Frédéric Péters il y a plus de 6 ans
Commencer la description du commit par "scss:" et pas "cells".
Si tout le titre devient cliquable, lui appliquer un cursor: pointer.
Mis à jour par Josué Kouka il y a plus de 6 ans
- Fichier 0001-scss-add-class-for-folded-and-foldable-cells-19496.patch 0001-scss-add-class-for-folded-and-foldable-cells-19496.patch ajouté
Frédéric Péters a écrit :
Les autres descriptions se terminent pas des points. "le contenu d'un cellule " → d'unE. AssociéE vu qu'on parle d'unE classe. "cette classe définit" : pour suivre les autres points on écrirait plutôt simplement "définit".
Fautes corrigés
Si tout le titre devient cliquable, lui appliquer un cursor: pointer.
Ajouté.
Mis à jour par Frédéric Péters il y a plus de 6 ans
- Fichier 0001-cells-add-class-for-folded-and-foldable-cells-19496.patch 0001-cells-add-class-for-folded-and-foldable-cells-19496.patch ajouté
Désormais testé et ça ne marche notamment pas sur les cellules de type texte (dont le balisage ne contient pas de <div> supplémentaire après le titre).
Proposition (à tester) en pièce jointe; j'y modifie aussi le sélecteur de titre pour correspondre à celui utilisé dans #19467 et je supprime les propriétés dupliquées.
Mis à jour par Josué Kouka il y a plus de 6 ans
Frédéric Péters a écrit :
Désormais testé et ça ne marche notamment pas sur les cellules de type texte (dont le balisage ne contient pas de <div> supplémentaire après le titre).
Proposition (à tester) en pièce jointe; j'y modifie aussi le sélecteur de titre pour correspondre à celui utilisé dans #19467 et je supprime les propriétés dupliquées.
Ok je viens de tester avec ton patch et le contenu de la cellule text n'est pas caché si folded est appliqué.
Mis à jour par Frédéric Péters il y a plus de 6 ans
Ok je viens de tester avec ton patch et le contenu de la cellule text n'est pas caché si folded est appliqué.
Tu peux poster son markup ? Éventuellement débugguer un peu ? Je viens de vérifier ici et ça fonctionne.
Mis à jour par Josué Kouka il y a plus de 6 ans
Frédéric Péters a écrit :
Ok je viens de tester avec ton patch et le contenu de la cellule text n'est pas caché si folded est appliqué.
Tu peux poster son markup ? Éventuellement débugguer un peu ? Je viens de vérifier ici et ça fonctionne.
Au temps pour moi, je confirme que ça fonctionne. Désolé
Mis à jour par Josué Kouka il y a plus de 6 ans
- Statut changé de En cours à Résolu (à déployer)
- % réalisé changé de 0 à 100
commit 6f703bef0a5e0499a1ac3df50fc0c94d3b48c05f
Author: Josue Kouka <jkouka@entrouvert.com>
Date: Wed Oct 18 11:33:22 2017 +0200
cells: add class for folded and foldable cells (#19496)
Mis à jour par Josué Kouka il y a plus de 6 ans
Serghei Mihai a écrit :
Il faudrait garder l'auteur du patch ;)
Désolé, j'ai pas fait attention. J'ai fait un git am tout simple.
Mis à jour par Frédéric Péters il y a plus de 6 ans
(j'avais de mon côté fait attention et consciemment conservé Josué comme auteur, notamment il y a aussi la partie documentation)
Mis à jour par Frédéric Péters il y a plus de 5 ans
- Statut changé de Résolu (à déployer) à Fermé
cells: add class for folded and foldable cells (#19496)