Projet

Général

Profil

Development #54143

Thème pour l'Eure-et-Loir (CD28)

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

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
20 mai 2021
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non


Fichiers


Demandes liées

Lié à Intégrations graphiques Publik - Development #57950: Thème pour l'Eure-et-Loir (CD28), ajout d'icône par catégorieFermé18 octobre 2021

Actions

Révisions associées

Révision ab9428c7 (diff)
Ajouté par Nicolas Roche il y a presque 3 ans

eurelien-cd28: add theme (#54143)

Historique

#3

Mis à jour par Brice Mallet il y a presque 3 ans

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

OK pour moi sur les aspects visuels / respect de ma demande initiale (https://dev.entrouvert.org/projects/cd28/wiki/Int%C3%A9gration_graphique)

#4

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

Mes remarques

  • Pourquoi ne pas utiliser les variable $nav-item-selected-background et $nav-button-background ?
  • simplifier
        .icon-bar {
    -        border: 2px solid $nav-button-color;
    +        border-width: 2px;
        }
    
  • Attention au design du menu mobile. Certains styles à n'appliquer que pour desktop.
        div.gru-nav {
            background-color: $secondary-color;
            ul {
                > li {
    +                @media (min-width: $nav-mobile-limit + 1) {
                        border-left: 2px solid white;
                        border-right: 2px solid white;
    +                }
                    &.selected {
                        background-color: $primary-color;
                    }
                    &:not(.selected) {
                        background-color: $nav-background;
                    }
                    > a {
                        font-size: 19px;
                        font-weight: 400;
    +                    padding-top: 0.33em;
    +                    padding-bottom: 0.33em;
    +                    @media (min-width: $nav-mobile-limit + 1) {
                            line-height: 36px;
                            padding: 0 15px;
    +                    }
                    }
                }
            }
        }
    
#5

Mis à jour par Nicolas Roche il y a presque 3 ans

Pourquoi ne pas utiliser les variable $nav-item-selected-background et $nav-button-background ?

C'est pour afficher la bande grise qui déborde à gauche et à droite des entrées du menu, y compris lorsque tous les élément ne couvrent pas toute la largeur du contenu (comme sur la caprture d'écran desktop).
Le but étant de me rapprocher de https://eurelien.fr/ (en desktop et mobile), s'il existe une technique plus propre, je prends.

simplifier

Bof, les barres du menu ne s'affichent plus.

Certains styles à n'appliquer que pour desktop.

J'ai simplifié.

#6

Mis à jour par Nicolas Roche il y a presque 3 ans

  • Fichier 0001-eurelien-cd28-add-theme-54143.patch ajouté

Bof, les barres du menu ne s'affichent plus.

j'ai dit de la ***

(patch qui utilise la variable $nav-item-selected-background entre autres)

#8

Mis à jour par Nicolas Roche il y a presque 3 ans

  • Fichier 0001-eurelien-cd28-add-theme-54143.patch supprimé
#9

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

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

Mis à jour par Nicolas Roche il y a presque 3 ans

  • Statut changé de Solution validée à Résolu (à déployer)
commit ab9428c7b99f9160954972ca5e6cc3d7839279d6
Author: Nicolas ROCHE <nroche@entrouvert.com>
Date:   Thu May 20 18:45:12 2021 +0200

    eurelien-cd28: add theme (#54143)
#11

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

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

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

  • Lié à Development #57950: Thème pour l'Eure-et-Loir (CD28), ajout d'icône par catégorie ajouté

Formats disponibles : Atom PDF