Development #12553
avoir une option séparée pour les classes CSS à appliquer à une cellule, plutôt que d'abuser du slug
0%
Description
En plus ça permettrait de gérer plusieurs classes.
Fichiers
Demandes liées
Révisions associées
Historique
Mis à jour par Frédéric Péters il y a presque 8 ans
- Lié à Development #12404: Template de pages plus souple ajouté
Mis à jour par Frédéric Péters il y a presque 8 ans
- Assigné à mis à Jean-Baptiste Jaillet
JB, l'idée est plutôt simple, un attribut extra_css_class (models.CharField) supplémentaire dans CellBase, son apparition dans la boite de dialogue "options" (PageCellOptionsView) et puis modifier la propriété css_class_name pour y ajouter la valeur de extra_css_class.
Mis à jour par Jean-Baptiste Jaillet il y a presque 8 ans
- Fichier 0001-manage-Add-a-new-field-for-extra-css-class-on-BaseCe.patch ajouté
- Statut changé de Nouveau à En cours
J'ai ajouté le champs.
Mis à jour par Thomas Noël il y a presque 8 ans
Il manque la migration, à inclure dans le même patch.
Et je ne ferais pas le « ... + ' ' + ... » si self.css_extra_class est vide, ni le lower().
Enfin au niveau du CharField, le nommer « Extra classes for CSS styling » comme dans wcs, histoire de.
Mis à jour par Frédéric Péters il y a presque 8 ans
Et je ne ferais pas le « ... + ' ' + ... » si self.css_extra_class est vide, ni le lower().
Oui, pas de .lower(), ce qui est tapé par l'usager doit être pris tel quel.
Enfin au niveau du CharField, le nommer « Extra classes for CSS styling » comme dans wcs, histoire de.
Et dans le même mimétisme, appeler ça extra_css_class.
Mis à jour par Frédéric Péters il y a presque 8 ans
fields=['slug', 'css_extra_class'],)
Et sur le patch, pas fan de laisser des virgules trainer derrière le dernier paramètre.
Mis à jour par Frédéric Péters il y a presque 8 ans
[manage]: Add a new field for extra css class on BaseCell
Et sur le message de commit, pas de [] autour du préfixe. (simplement "manage: ...") (et pas de majuscule non plus au premier mot)
Mis à jour par Jean-Baptiste Jaillet il y a presque 8 ans
- Fichier
0001-manage-Add-a-new-field-for-extra-css-class-on-BaseCe.patchsupprimé
Mis à jour par Jean-Baptiste Jaillet il y a presque 8 ans
- Fichier 0001-manage-add-a-new-field-for-extra-css-class-on-BaseCe.patch 0001-manage-add-a-new-field-for-extra-css-class-on-BaseCe.patch ajouté
J'ai fait les modifications
Mis à jour par Frédéric Péters il y a plus de 7 ans
- Fichier 0001-manage-add-a-new-field-for-extra-css-class-on-BaseCe.patch 0001-manage-add-a-new-field-for-extra-css-class-on-BaseCe.patch ajouté
- Il manque la migration, à inclure dans le même patch.
- Et... ni le lower().
- Et dans le même mimétisme, appeler ça extra_css_class.
Ces trois points n'étaient pas pris en compte, patch actualisé, avec un message sans le bruit de détails. (et avec un tas de migrations, qui suivent master + #12758).
Mis à jour par Frédéric Péters il y a plus de 7 ans
- Fichier 0001-manage-add-a-new-field-for-extra-css-class-on-BaseCe.patch 0001-manage-add-a-new-field-for-extra-css-class-on-BaseCe.patch ajouté
Correction supplémentaire pour que le nouveau champ n'apparaisse pas dans les champs de la cellule dépliée.
Mis à jour par Frédéric Péters il y a plus de 7 ans
- Statut changé de En cours à Résolu (à déployer)
commit 9242e4d37a16a3c956f078b75538276dfd4da0a7
Author: Frédéric Péters <fpeters@entrouvert.com>
Date: Wed Jul 13 17:47:15 2016 +0200
manage: add a new field for extra css class on BaseCell (#12553)
Mis à jour par Frédéric Péters il y a plus de 5 ans
- Statut changé de Résolu (à déployer) à Solution déployée
manage: add a new field for extra css class on BaseCell (#12553)