Development #50094
supprimer class .large posée sur #content
0%
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
themes: remove useless .large class on #content (#50094)
Historique
Mis à jour par Thomas Jund il y a environ 3 ans
- Fichier 0001-themes-remove-useless-.large-class-on-content-50094.patch 0001-themes-remove-useless-.large-class-on-content-50094.patch ajouté
- Fichier 0001-themes-remove-useless-.large-class-on-content-50094.patch 0001-themes-remove-useless-.large-class-on-content-50094.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Patch publik-base-theme et atreal-publik-themes.
Rien à signalé coté imio et rouen.
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 {
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.
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.
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.
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...)
Mis à jour par Thomas Jund il y a environ 3 ans
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)
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)
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
themes: remove useless .large class on #content (#50094)