Projet

Général

Profil

Development #24448

réduction de la spécificité des CSS partagées : #content

Ajouté par Frédéric Péters il y a presque 6 ans. Mis à jour il y a plus de 5 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Frédéric Péters
Version cible:
-
Début:
11 juin 2018
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:

Description

cf #24446 pour le raisonnement, ici la zone de contenu de la page. Parce que class="content" est déjà utilisé par ailleurs, et pour suivre ce qui existait déjà à Alfortville et Nancy, utilisation d'une classe "gru-content".

Fichiers

0001-scss-use-a-class-selector-to-reference-page-content-.patch (12,4 ko) 0001-scss-use-a-class-selector-to-reference-page-content-.patch Frédéric Péters, 11 juin 2018 21:24

Demandes liées

Lié à Intégrations graphiques Publik - Development #24446: réduction de la spécificité des CSS partagéesNouveau11 juin 2018

Actions

Révisions associées

Révision 71556948 (diff)
Ajouté par Frédéric Péters il y a presque 6 ans

scss: use a class selector to reference page #content (#24448)

Historique

#1

Mis à jour par Frédéric Péters il y a presque 6 ans

#2

Mis à jour par Frédéric Péters il y a presque 6 ans

#3

Mis à jour par Anonyme il y a presque 6 ans

C'est un très très bonne idée de réduire la spécificité.
Je propose une lecture de ton patch.

  • comme c'est le cas dans le patch :
    • il y a les sélecteurs avec un nom d'élement HTML, qui sont précédés par .gru-content: div.gru-content button
    • il y a des sélecteurs avec une classe au nom très générique, qui sont précédés par .gru-content : .gru-content #columns > .cell, .gru-@div.textcell h2:first-child,
  • mais, est-ce qu'il n'y a pas certains sélecteurs qui sont inutilement précédés d'un .gru-content ? par exemple :
    • .gru-content #un-identifiant-précis: #disclose-dataview seulement ?
    • .gru-content div.cell.une-classe-précise: .cell.wcsformsofcategorycell seulement ?
  • est-ce qu'on pourrait aussi profiter pour éliminer les div.gru-content qui ne baisse pas la spécificité d'un nom d'élément avec une classe qui force à répéter partout où on surchargera le nom de l'élément : div.gru-content form.disabled-during-submit
  • comme c'est une modification générale, est-ce qu'il ne faudrait pas des captures avec les thèmes existants impactés ?
#4

Mis à jour par Frédéric Péters il y a presque 6 ans

Je trouve important de limiter au maximum les modifications, pour limiter les risques de conséquence sur les intégrations; et donc non, de ne pas en profiter pour changer quoique ce soit d'autre. (il sera temps, plus tard).

comme c'est une modification générale, est-ce qu'il ne faudrait pas des captures avec les thèmes existants impactés ?

J'ai vérifié Strasbourg et Alfortville qui sont deux intégrations hors des clous.

#5

Mis à jour par Anonyme il y a presque 6 ans

ok pour moi

#6

Mis à jour par Frédéric Péters il y a presque 6 ans

  • Statut changé de En cours à Résolu (à déployer)
commit 7155694831a2d3f155b7bc658dc0be1b49ed936b
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Mon Jun 11 21:20:16 2018 +0200

    scss: use a class selector to reference page #content (#24448)
#7

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

Formats disponibles : Atom PDF