Projet

Général

Profil

Development #42868

que l'option foldable ne masque plus les images

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

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

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

L'option `.foldable` masque les images de la cellule par défaut, sauf si l'option $cell-image-position == top.

Comme noté https://dev.entrouvert.org/issues/41741#note-13

Je ne comprends pas trop le pourquoi de ce fonctionnement, l'utilisation d'une image a pour objectif de facilité l'identification des catégories ou du moins d'illustrer le titre, si elle est masquée qu'elle est son intérêt ?

Je propose que l'option foldable ne masque plus par défaut l'image de la cellule quel que soit sa position.


Fichiers

Révisions associées

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

css: don't hide cell picture when cell has foldable option (#42868)

Historique

#1

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

#2

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

Au grand jeu de l'existant, ça concerne donc toutes les intégrations sauf celles qui mentionnent $cell-image-position: top; mais toutes n'utilisent pas d'image, ni de foldable, mais tu as fait un tour voir si ça arrive quand même ? (et confirmer que ce sera mieux ainsi)

(ok pour moi si c'est fait en considérant l'existant)

#3

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

  • Statut changé de Solution proposée à En cours

Je vais prendre le temps de sonder encore un peu, mais je n'ai jamais rencontré sur un thème la combinaison "foldable and image and not `$cell-image-position = top`".

#4

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

  • Statut changé de En cours à Solution proposée

Grâce à l'export .json des combos :

  1. j'ai filtré les thèmes qui ont chargé des assets dans les catégories WCS,
  2. filtré les thèmes qui utilisent la class foldable
  3. vérifié si ces thèmes utilisent la var sass `$cell-image-position = top`
  4. checké le code des thèmes restants pour voir si l'image est masquée losrqu'une cellule est repliée

Je n'ai trouvé aucun thème dont l'image d'une cellule serait masquée lorsque le cellule est repliée.

#5

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

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

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

  • Statut changé de Solution validée à Résolu (à déployer)
commit e775f2eee5fc724be931cf436cff2b0fd03c963a (HEAD -> main, origin/main)
Author: Thomas JUND <tjund@entrouvert.com>
Date:   Thu May 14 12:30:05 2020 +0200

    css: don't hide cell picture when cell has foldable option (#42868)
#7

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