Development #48515
Development #44934: Plan amélioration layout des templates
remplacer #left et #right par .column
0%
Description
Fait suite à #47913.
Remplacer les id #left et #right par une class unique .column.L'idée étant de pouvoir
- utiliser ce système de layout facilement à d'autres endroits
- avec un nombre plus large de colonnes.
- éviter la duplication des id #left et #right en cas d'imbrication des pages a2 dans un template à multiples colonnes.
- réduire la spécificité des sélecteurs.
Fichiers
Révisions associées
hautes-alpes-2018: introduce .column instead #left & #right (#48515)
alpes-maritimes-2018: introduce .column instead #left & #right (#48515)
amiens-metropole: introduce .column instead #left & #right (#48515)
arpajon: introduce .column instead #left & #right (#48515)
grenoble-metropole-2019: introduce .column instead #left & #right (#48515)
groupe-up: introduce .column instead #left & #right (#48515)
lille: introduce .column instead #left & #right (#48515)
montpellier: introduce .column instead #left & #right (#48515)
roannais-icitoyen: introduce .column instead #left & #right (#48515)
saint-chamond: introduce .column instead #left & #right (#48515)
seine-et-marne-apa: introduce .column instead #left & #right (#48515)
groupe-up: introduce .column instead #left & #right (#48515)
signal-publik: introduce .column instead #left & #right (#48515)
villeneuve-dascq: introduce .column instead #left & #right (#48515)
mincult: introduce .column instead #left & #right (#48515)
rouen: introduce .column instead of #left & #right (#48515)
Historique
Mis à jour par Thomas Jund (congés, retour le 29/04) il y a plus de 3 ans
- Fichier 0001-templates-introduce-.column-instead-left-right-48515.patch 0001-templates-introduce-.column-instead-left-right-48515.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Patch pour le core.
14 thèmes patchés dans la branch
Mis à jour par Nicolas Roche il y a plus de 3 ans
J'ose pas valider, mais ça me semble aussi une bonne idée de remplacer #left/#right
par .first-child/.last-child
Mis à jour par Frédéric Péters il y a plus de 3 ans
Nicolas, le truc que tu peux faire c'est appliquer la branche et vivre avec quelques jours, comme ça vendredi prochain tu pourras dire que tu n'as rien vu de cassé. (ou tu auras vu des trucs cassés et ça aura pu être corrigé).
Mis à jour par Thomas Jund (congés, retour le 29/04) il y a plus de 3 ans
- Statut changé de Solution proposée à En cours
C'est un patch transitoire, en point de mire, je souhaite arriver à une solution simple ou .columns est un module de template qui juxtapose cote à cote ses enfants .column (tous la même taille)
.columns .column .column
Pouf 2 colonnes de 50%
.columns .column .column .column
Pouf 3 colonnes de 33.333…%
et pouvoir faire du nesting
.columns .column .columns .column .column .column
voir ticket parent #44934
J'y vais tout doucement, parce que tout le code est trop fragile.
J'ai gardé #left et #right pour le moment, dans le doute pour les templates très specifiques.
J'ose pas valider, mais ça me semble aussi une bonne idée de remplacer #left/#right par .first-child/.last-child
CSS propose maintenant tout ce qu'il faut pour cibler directement le premier, deuxième, ou nth element, je ne vois pas trop l'intérêt d'ajouter des class CSS pour cela. Mais peut-être que je ne vois pas ton cas d'usage ?
Je repasse le ticket en "en cours", parce que je n'ai pas fini de regarder les thèmes imio, rouen, atreal.
@nico, comme le propose Fred, je suis preneur d'un deuxième regard parce qu'il est vite arrivé de passer à côté d'un thème "cassé"
Mis à jour par Thomas Jund (congés, retour le 29/04) il y a plus de 3 ans
- Fichier 0001-templates-introduce-.column-instead-left-right-48515.patch 0001-templates-introduce-.column-instead-left-right-48515.patch ajouté
Patch themes imio
Mis à jour par Thomas Jund (congés, retour le 29/04) il y a plus de 3 ans
- Fichier 0001-templates-introduce-.column-instead-left-right-48515.patch 0001-templates-introduce-.column-instead-left-right-48515.patch ajouté
- Fichier 0001-templates-introduce-.column-instead-left-right-48515.patch 0001-templates-introduce-.column-instead-left-right-48515.patch ajouté
- Fichier 0001-templates-introduce-.column-instead-left-right-48515.patch 0001-templates-introduce-.column-instead-left-right-48515.patch ajouté
- Statut changé de En cours à Solution proposée
Dernier round :
- Après avoir vérifié d'avoir supprimé toutes les références à #left et #right, j'ai supprimé les id dans les templates, pour ne pas risquer de les voir réapparaître.
- Patch de mincult qui j'avais zappé
- Update du patch de villeneuve-dascq où j'en profite pour supprimer des styles devenues inutiles dans le layout 3 cols.
et J'avais oublié 2 lignes pour imio
Je pose le patch pour Rouen et rien à signaler du côté des thèmes atreal.
Les patchs des thèmes "publik-base-theme" disponibles dans la branch
Ready for ack
Mis à jour par Frédéric Péters il y a plus de 3 ans
- Statut changé de Solution proposée à Solution validée
Pas véritablement testé on verra à l'usage.
Mis à jour par Thomas Jund (congés, retour le 29/04) il y a plus de 3 ans
- Statut changé de Solution validée à Résolu (à déployer)
Patchs publik-base-theme
commit 7b8c35d337e1ca8cb6885c2c2fc80f63b66c9db2 (HEAD -> main, origin/main, wip/48515-class-column) Author: Thomas JUND <tjund@entrouvert.com> Date: Wed Jan 13 14:24:26 2021 +0100 mincult: introduce .column instead #left & #right (#48515) commit 30aaa783c7fd50a454b561702dda86846bd02d74 Author: Thomas JUND <tjund@entrouvert.com> Date: Wed Nov 11 12:16:30 2020 +0100 villeneuve-dascq: introduce .column instead #left & #right (#48515) commit 9ac530a810bef873799d905d16bc22287e5e5825 Author: Thomas JUND <tjund@entrouvert.com> Date: Wed Nov 11 12:16:05 2020 +0100 signal-publik: introduce .column instead #left & #right (#48515) commit 5a5a1e80fd17070026938a7f6e6d6819d55cfd91 Author: Thomas JUND <tjund@entrouvert.com> Date: Wed Nov 11 12:13:16 2020 +0100 groupe-up: introduce .column instead #left & #right (#48515) commit aa0eb3857151fa87d0f462e643bead2ed1cf46eb Author: Thomas JUND <tjund@entrouvert.com> Date: Wed Nov 11 12:09:06 2020 +0100 seine-et-marne-apa: introduce .column instead #left & #right (#48515) commit 34cc82526f83a3eda13abb7f002b885b832916ba Author: Thomas JUND <tjund@entrouvert.com> Date: Tue Nov 10 12:19:34 2020 +0100 saint-chamond: introduce .column instead #left & #right (#48515) commit 7525712e6af3fcc93dee43e83f61bae7300eefe1 Author: Thomas JUND <tjund@entrouvert.com> Date: Tue Nov 10 12:06:16 2020 +0100 roannais-icitoyen: introduce .column instead #left & #right (#48515) commit cfdb6db330ca22d41653c3c4724c89bd69dca778 Author: Thomas JUND <tjund@entrouvert.com> Date: Tue Nov 10 12:01:51 2020 +0100 montpellier: introduce .column instead #left & #right (#48515) commit e1bfa20d745e54204970bccdcbd55e8f8f0a5325 Author: Thomas JUND <tjund@entrouvert.com> Date: Tue Nov 10 11:47:47 2020 +0100 lille: introduce .column instead #left & #right (#48515) commit 6cb8fe7cc70fe4c130aa7f875bc7407c55c9e042 Author: Thomas JUND <tjund@entrouvert.com> Date: Mon Nov 9 18:04:21 2020 +0100 groupe-up: introduce .column instead #left & #right (#48515) commit be6be2d0a36f5c35d31a36a44ac8e84ff8bad0a6 Author: Thomas JUND <tjund@entrouvert.com> Date: Mon Nov 9 18:03:52 2020 +0100 grenoble-metropole-2019: introduce .column instead #left & #right (#48515) commit ad351257f861d3d56ae561b243a12593121d292b Author: Thomas JUND <tjund@entrouvert.com> Date: Mon Nov 9 17:41:15 2020 +0100 arpajon: introduce .column instead #left & #right (#48515) commit 1d7b967bf3f9121643eabeacfc8d67793d725267 Author: Thomas JUND <tjund@entrouvert.com> Date: Mon Nov 9 17:26:34 2020 +0100 amiens-metropole: introduce .column instead #left & #right (#48515) commit 94f810dffa0690e95d0ea555bfdcf1e5266c419a Author: Thomas JUND <tjund@entrouvert.com> Date: Mon Nov 9 17:16:28 2020 +0100 alpes-maritimes-2018: introduce .column instead #left & #right (#48515) commit 34841965d36f8cbf74787e09b439e77832d40925 Author: Thomas JUND <tjund@entrouvert.com> Date: Fri Nov 13 11:16:51 2020 +0100 hautes-alpes-2018: introduce .column instead #left & #right (#48515) commit 04bdc5a8f5268201abf250f70303e5099f4be413 Author: Thomas JUND <tjund@entrouvert.com> Date: Fri Nov 13 11:15:16 2020 +0100 templates: introduce .column instead #left & #right (#48515)
Mis à jour par Thomas Jund (congés, retour le 29/04) il y a plus de 3 ans
Patch imio-publik-themes
commit c766e7522f06d234757d61d0a580d9d4b3e65c77 Author: Thomas JUND <tjund@entrouvert.com> Date: Wed Jan 13 12:23:43 2021 +0100 templates: introduce .column instead #left & #right (#48515)
Mis à jour par Thomas Jund (congés, retour le 29/04) il y a plus de 3 ans
Patch rouen-publik-theme
commit bd3d9adfabb45a8c37ac93319179f6fe79dc581f Author: Thomas JUND <tjund@entrouvert.com> Date: Wed Jan 13 12:34:56 2021 +0100 templates: introduce .column instead #left & #right (#48515)
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 Frédéric Péters il y a plus de 3 ans
Pour info #50453 côté imio, parce que façon de mettre à jour différente et donc à la fois publik-base-theme stable et imio-publik-themes de eobuilder; sur des changements de ce type pour éviter ça il faudrait soit étaler sur deux cycles (concrètement ici garder pendant un cycle dans le markup les id), soit penser à forcer une version supérieure dans le debian/control de imio-publik-themes.
templates: introduce .column instead #left & #right (#48515)