Projet

Général

Profil

Development #50094

supprimer class .large posée sur #content

Ajouté par Thomas Jund il y a environ 3 ans. Mis à jour il y a environ 3 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
13 janvier 2021
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

Dans theme.html, la balise #content possède une class `.large` qui me semble ne servir à rien.
Elle est parfois utilisée dans les thèmes pour enchérir uniquement le poids des sélecteur.
Je propose de la supprimer.


Fichiers

Révisions associées

Révision 53909fc7 (diff)
Ajouté par Thomas Jund il y a environ 3 ans

themes: remove useless .large class on #content (#50094)

Révision fe9dd8fa (diff)
Ajouté par Thomas Jund il y a environ 2 ans

themes: remove useless .large class on #content (#50094)

Historique

#1

Mis à jour par Thomas Jund il y a environ 3 ans

Patch publik-base-theme et atreal-publik-themes.
Rien à signalé coté imio et rouen.

#2

Mis à jour par Nicolas Roche (absent jusqu'au 3 avril) il y a environ 3 ans

J'ai l'impression que large n'a pas été introduit dans un but précis (il est là depuis le début) : https://git.entrouvert.org/publik-base-theme.git/commit/?id=c670af344ca8541f0e936025a926ca4ccc061f45

Et j'ai aussi l'impression qu'il reste une occurrence ici (mais je me demande si cette règle est vraiment utilisée) :

$ git grep '\.large'
static/metz-metropole-2019/_custom.scss:div.cell.large a.big-image-link {

#3

Mis à jour par Thomas Jund il y a environ 3 ans

J'ai l'impression que large n'a pas été introduit dans un but précis

Oui c'est pourquoi je propose de la supprimer

Et j'ai aussi l'impression qu'il reste une occurrence

Dans ce cas, la class est posée sur une .cell. Sûrement une class spécifique au thème. Rien à voir avec celle posée sur #content.

#4

Mis à jour par Nicolas Roche (absent jusqu'au 3 avril) il y a environ 3 ans

Sûrement une classe spécifique au thème.

Bof, je ne pense pas que ce soit utilisé par le thème.
Si c'est du code mort, je me demandais s'il ne fallait pas le supprimer dans la foulée.

#5

Mis à jour par Thomas Jund il y a environ 3 ans

Bof, je ne pense pas que ce soit utilisé par le thème.

En explorant l'export du combo prod de metz-metropole, 12 cellules de types liens (data.linkcell) utilisent la class .large au sein de la page signalement.

#6

Mis à jour par Nicolas Roche (absent jusqu'au 3 avril) il y a environ 3 ans

Je suis désolé d'insister pour quelque-chose qui n'a pas d'importance, mais je ne les vois pas :

$ ssh combo.node1.prod.saas.entrouvert.org
$ grep -R large /var/lib/combo/tenants/services.metzmetropole.fr/templates

(si tu pouvais me donner un lien...)

#8

Mis à jour par Nicolas Roche (absent jusqu'au 3 avril) il y a environ 3 ans

  • Statut changé de Solution proposée à Solution validée

Classes ajoutées via le backoffice du portail :

Classes supplémentaires pour le style par CSS : grid-1-4 large

(merci, j'ai compris)

#9

Mis à jour par Thomas Jund il y a environ 3 ans

  • Statut changé de Solution validée à Résolu (à déployer)
commit 53909fc744a111b95db373c29d342f7da1febf77 (HEAD -> main, origin/main)
Author: Thomas JUND <tjund@entrouvert.com>
Date:   Thu Jan 28 16:22:46 2021 +0100

    themes: remove useless .large class on #content (#50094)
#10

Mis à jour par Frédéric Péters il y a environ 3 ans

  • Statut changé de Résolu (à déployer) à Solution déployée

Formats disponibles : Atom PDF