Projet

Général

Profil

Bug #17818

css pour shown-because-admin

Ajouté par Thomas Noël il y a presque 7 ans. Mis à jour il y a plus de 5 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
-
Version cible:
-
Début:
27 juillet 2017
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:

Description

Combo affiche toutes ces cellules à l'admin du site.

Les cellules qui sont affichées alors qu'on n'a pas les rôles nécessaires ont une classe "shown-because-admin"

Pour les frontoffice, c'est différencié via publik-base-theme :

#content div.cell.shown-because-admin {
        h2 {
                background: transparent;
        }
        opacity: 0.5;
        background-image: repeating-linear-gradient(-45deg, #eee 0px, #eee 14px, transparent 15px, transparent 30px);
        &:hover {
                h2 {
                        background: inherit;
                }
                opacity: 1;
                background-image: inherit;
        }
}

En backoffice il n'y a rien pour cette classes, et je crois que ça devrait faire partie de gadjo.


Fichiers

Révisions associées

Révision 657e1ace (diff)
Ajouté par Thomas Noël il y a presque 7 ans

css: add shown-because-admin class, combo specific (#17818)

Historique

#1

Mis à jour par Thomas Noël il y a presque 7 ans

  • Fichier 0001-css-add-shown-because-admin-class-combo-specific-178.patch ajouté
  • Statut changé de Nouveau à En cours
  • Patch proposed changé de Non à Oui
#3

Mis à jour par Thomas Noël il y a presque 7 ans

  • Fichier 0001-css-add-shown-because-admin-class-combo-specific-178.patch supprimé
#4

Mis à jour par Thomas Noël il y a presque 7 ans

  • Patch proposed changé de Oui à Non

Ah non, ne passe pas pour mon cas où les cellules sont dans la sidebar ... (est-ce pareil dans publik-base-theme ?)

#5

Mis à jour par Thomas Noël il y a presque 7 ans

Voilà une version qui pose un style pour div.cell.shown-because-admin en général... Ça me parait plus générique, mais pas trop non plus.

#6

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

Je pense qu'il n'y a pas besoin d'ajouter des règles pour h2, patch attaché, pour moi ça serait ok.

#7

Mis à jour par Thomas Noël il y a presque 7 ans

  • Statut changé de En cours à Résolu (à déployer)

Merci

commit 657e1ace147fcfd5dfb6e4a5e8a0bb9f71ca5884
Author: Thomas NOEL <tnoel@entrouvert.com>
Date:   Thu Jul 27 12:10:00 2017 +0200

    css: add shown-because-admin class, combo specific (#17818)

#8

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