Projet

Général

Profil

Development #48009

Thème du Ministère des Solidarités et de la Santé

Ajouté par Nicolas Roche il y a plus de 3 ans. Mis à jour il y a plus de 3 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Brice Mallet
Version cible:
-
Début:
23 octobre 2020
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

Il a été convenu de se baser sur l'exemple du Ministère de la Culture et sur cette copie d'écran de l'intranet du ministère "PACo".

Fichiers

Paco.jpg (337 ko) Paco.jpg Nicolas Roche, 23 octobre 2020 16:57
0001-add-minsant-theme-48009.patch (29,6 ko) 0001-add-minsant-theme-48009.patch Nicolas Roche, 23 octobre 2020 17:33
Screenshot_2020-10-23 Portail - Accueil(1).png (256 ko) Screenshot_2020-10-23 Portail - Accueil(1).png Nicolas Roche, 23 octobre 2020 17:33
Screenshot_2020-10-23 Portail - Accueil.png (397 ko) Screenshot_2020-10-23 Portail - Accueil.png Nicolas Roche, 23 octobre 2020 17:33

Révisions associées

Révision 55fa1bad (diff)
Ajouté par Nicolas Roche il y a plus de 3 ans

add minsant theme (#48009)

Historique

#4

Mis à jour par Brice Mallet il y a plus de 3 ans

Je valide (graphiquement) l'intégration réalisée, je te laisse voir avec qui tu veux pour la validation technique

#5

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

  • Statut changé de Solution proposée à Solution validée
- div#right.account-management {
+ .account-management {
    display: none;
}

Je le plan de virer les id #right et #left d'ici très bientôt dans ma refacto du layout (en premier ceux des pages authentic).
Cibler l’élément uniquement avec `.account-management` devrait suffire.
De manière globale, moins tes sélecteurs ont d'information sur l'arbre DOM, mieux c'est.

Et petite best practice au passage.
Il est préférable de déclarer tes couleurs par une variable explicite (définir un nuancier) et d'ensuite de les associer à $primary/$secondary-color. Ça permet de gagner en clarté à la relecture du code surtout quand on utilise les couleurs en valeur hexa.

$gray-light: #f3f3f3;
$blue-dark: #697b98;

$primary-color: $gray-light;
$secondary-color: $blue-dark;

Je valide, on est sur un thème ultra-natif :)

#6

Mis à jour par Nicolas Roche il y a plus de 3 ans

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

En fait il ne fallait pas masquer le pavé "account-management" ici, du coup merci pour ta remarque.
Et j'ai donc défini mon nuancier.

commit 55fa1bad4599d0bf6001e3ed5185898b16f3270e
Author: Nicolas ROCHE <nroche@entrouvert.com>
Date:   Fri Oct 23 17:01:37 2020 +0200

    add minsant theme (#48009)
#7

Mis à jour par Brice Mallet il y a plus de 3 ans

Je pensais que ça passerait en recette ce jeudi 05/11 soir (c'est pas urgent donc pas grave mais comme était possible, je suis un peu déçu;-)

#8

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

C'est désormais disponible en recette.

#9

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

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

Mis à jour par Brice Mallet il y a plus de 3 ans

  • Assigné à changé de Nicolas Roche à Brice Mallet

Frédéric Péters a écrit :

C'est désormais disponible en recette.

Merci Fred

Formats disponibles : Atom PDF