Projet

Général

Profil

Development #48515

Development #44934: Plan amélioration layout des templates

remplacer #left et #right par .column

Ajouté par Thomas Jund (congés, retour le 29/04) il y a plus de 3 ans. Mis à jour il y a plus de 3 ans.

Statut:
Fermé
Priorité:
Normal
Version cible:
-
Début:
13 novembre 2020
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

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

Révision 04bdc5a8 (diff)
Ajouté par Thomas Jund (congés, retour le 29/04) il y a plus de 3 ans

templates: introduce .column instead #left & #right (#48515)

Révision 34841965 (diff)
Ajouté par Thomas Jund (congés, retour le 29/04) il y a plus de 3 ans

hautes-alpes-2018: introduce .column instead #left & #right (#48515)

Révision 94f810df (diff)
Ajouté par Thomas Jund (congés, retour le 29/04) il y a plus de 3 ans

alpes-maritimes-2018: introduce .column instead #left & #right (#48515)

Révision 1d7b967b (diff)
Ajouté par Thomas Jund (congés, retour le 29/04) il y a plus de 3 ans

amiens-metropole: introduce .column instead #left & #right (#48515)

Révision ad351257 (diff)
Ajouté par Thomas Jund (congés, retour le 29/04) il y a plus de 3 ans

arpajon: introduce .column instead #left & #right (#48515)

Révision be6be2d0 (diff)
Ajouté par Thomas Jund (congés, retour le 29/04) il y a plus de 3 ans

grenoble-metropole-2019: introduce .column instead #left & #right (#48515)

Révision 6cb8fe7c (diff)
Ajouté par Thomas Jund (congés, retour le 29/04) il y a plus de 3 ans

groupe-up: introduce .column instead #left & #right (#48515)

Révision e1bfa20d (diff)
Ajouté par Thomas Jund (congés, retour le 29/04) il y a plus de 3 ans

lille: introduce .column instead #left & #right (#48515)

Révision cfdb6db3 (diff)
Ajouté par Thomas Jund (congés, retour le 29/04) il y a plus de 3 ans

montpellier: introduce .column instead #left & #right (#48515)

Révision 7525712e (diff)
Ajouté par Thomas Jund (congés, retour le 29/04) il y a plus de 3 ans

roannais-icitoyen: introduce .column instead #left & #right (#48515)

Révision 34cc8252 (diff)
Ajouté par Thomas Jund (congés, retour le 29/04) il y a plus de 3 ans

saint-chamond: introduce .column instead #left & #right (#48515)

Révision aa0eb385 (diff)
Ajouté par Thomas Jund (congés, retour le 29/04) il y a plus de 3 ans

seine-et-marne-apa: introduce .column instead #left & #right (#48515)

Révision 5a5a1e80 (diff)
Ajouté par Thomas Jund (congés, retour le 29/04) il y a plus de 3 ans

groupe-up: introduce .column instead #left & #right (#48515)

Révision 9ac530a8 (diff)
Ajouté par Thomas Jund (congés, retour le 29/04) il y a plus de 3 ans

signal-publik: introduce .column instead #left & #right (#48515)

Révision 30aaa783 (diff)
Ajouté par Thomas Jund (congés, retour le 29/04) il y a plus de 3 ans

villeneuve-dascq: introduce .column instead #left & #right (#48515)

Révision 7b8c35d3 (diff)
Ajouté par Thomas Jund (congés, retour le 29/04) il y a plus de 3 ans

mincult: introduce .column instead #left & #right (#48515)

Révision 5e78be01 (diff)
Ajouté par Thomas Jund (congés, retour le 29/04) il y a environ 2 ans

rouen: introduce .column instead of #left & #right (#48515)

Historique

#1

Mis à jour par Thomas Jund (congés, retour le 29/04) il y a plus de 3 ans

Patch pour le core.

14 thèmes patchés dans la branch

#2

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

#3

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é).

#4

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é"

#6

Mis à jour par Thomas Jund (congés, retour le 29/04) il y a plus de 3 ans

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

#7

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.

#8

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)
#9

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)
#10

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)
#11

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
#12

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.

Formats disponibles : Atom PDF