Project

General

Profile

Development #54143

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

Added by Nicolas Roche 8 months ago. Updated 8 months ago.

Status:
Solution déployée
Priority:
Normal
Assignee:
Target version:
-
Start date:
20 May 2021
Due date:
% Done:

0%

Estimated time:
Patch proposed:
Yes
Planning:
No


Files


Related issues

Related to Intégrations graphiques Publik - Development #57950: Thème pour l'Eure-et-Loir (CD28), ajout d'icône par catégorieSolution déployée18 Oct 2021

Actions

Associated revisions

Revision ab9428c7 (diff)
Added by Nicolas Roche 8 months ago

eurelien-cd28: add theme (#54143)

History

#3

Updated by Brice Mallet 8 months ago

  • Status changed from Solution proposée to 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

Updated by Thomas Jund 8 months ago

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

Updated by Nicolas Roche 8 months ago

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

Updated by Nicolas Roche 8 months ago

  • File 0001-eurelien-cd28-add-theme-54143.patch added

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

Updated by Nicolas Roche 8 months ago

  • File deleted (0001-eurelien-cd28-add-theme-54143.patch)
#9

Updated by Thomas Jund 8 months ago

  • Status changed from Solution proposée to Solution validée
#10

Updated by Nicolas Roche 8 months ago

  • Status changed from Solution validée to 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

Updated by Frédéric Péters 8 months ago

  • Status changed from Résolu (à déployer) to Solution déployée
#12

Updated by Brice Mallet 3 months ago

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

Also available in: Atom PDF