Projet

Général

Profil

Bug #38681

correction sur l'intégration graphique pour la ville de saint-denis (93)

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

Statut:
Fermé
Priorité:
Normal
Assigné à:
Nicolas Roche
Version cible:
-
Début:
21 décembre 2019
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

Corrections suite à la mise en recette (suite de #38589) :
les éléments de la navigation sont bleu moche,
le titre du site se superpose au logo, etc.

Fichiers

0001-saint-denis-hide-logo-title-and-correct-menu-38681.patch (1,52 ko) 0001-saint-denis-hide-logo-title-and-correct-menu-38681.patch Nicolas Roche, 21 décembre 2019 21:55
Screenshot.png (61,5 ko) Screenshot.png Nicolas Roche, 21 décembre 2019 21:56
0001-saint-denis-nav-colors-proposition.patch (1,65 ko) 0001-saint-denis-nav-colors-proposition.patch Thomas Jund, 02 janvier 2020 16:27
0002-saint-denis-active-background-size-cover-on-page-ima.patch (1,73 ko) 0002-saint-denis-active-background-size-cover-on-page-ima.patch Thomas Jund, 02 janvier 2020 16:27
0003-saint-denis-drag-page-image-under-nav.patch (1,13 ko) 0003-saint-denis-drag-page-image-under-nav.patch Thomas Jund, 02 janvier 2020 16:27
Screenshot_2020-01-03 Portail - Test.png (20,2 ko) Screenshot_2020-01-03 Portail - Test.png Nicolas Roche, 03 janvier 2020 11:16
0001-saint-denis-correct-nav-elements-38681.patch (2,32 ko) 0001-saint-denis-correct-nav-elements-38681.patch Nicolas Roche, 03 janvier 2020 11:16
0001-saint-denis-correct-nav-elements-38681.patch (2,52 ko) 0001-saint-denis-correct-nav-elements-38681.patch Nicolas Roche, 10 janvier 2020 13:34
0001-saint-denis-correct-nav-elements-38681.patch (19,9 ko) 0001-saint-denis-correct-nav-elements-38681.patch Nicolas Roche, 10 janvier 2020 14:38

Demandes liées

Lié à Intégrations graphiques Publik - Development #38353: intégration graphique pour la ville de saint-denis (93)Fermé10 décembre 2019

Actions

Révisions associées

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

saint-denis: correct nav elements (#38681)

Historique

#1

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

Patch qui :
  • retire le texte sur le logo : Demo Saint-Denis (93)
    (étrange, je ne l'avais pas en local)
  • remonte le menu de quelques pixels
  • utilise du gris clair pour les items non selectionés dans le menu

Sur ce dernier point, je veux bien ton avis Thomas, stp.

#2

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

  • Lié à Development #38353: intégration graphique pour la ville de saint-denis (93) ajouté
#3

Mis à jour par Thomas Jund il y a plus de 4 ans

  • 0001-saint-denis-nav-colors-proposition
    Je propose une autre version pour les couleurs de la navigation.
    En utilisant la couleur primaire juste pour l'élément actif
  • 0002-saint-denis-active-background-size-cover-on-page-ima
    Ce patch redéfini la taille de l'image atfer-nav pour pouvoir utiliser un background-size cover dessus (plus adapté en responsive) et avec une marge bottom grise constante.
  • 0003-saint-denis-drag-page-image-under-nav
    Patch un peu plus fun où je propose de glisser l'imager sous la nav

Je te laisse voir ce que tu merges.

#4

Mis à jour par Thomas Jund il y a plus de 4 ans

  • Assigné à changé de Thomas Jund à Nicolas Roche
#5

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

Merci, j'ai appliqué 0001 et 0002.
Je trouve que c'est beaucoup plus léger comme ça.

#6

Mis à jour par Serghei Mihai il y a plus de 4 ans

Pourquoi letter-spacing: -0.03em;.
Sur ta capture les lettres ont l'air trop collés.

#7

Mis à jour par Thomas Jund il y a plus de 4 ans

C'est moi qui ai proposé ce style typo, pour mieux s'approcher, alors qu'on utilise une police websafe, du style du site de Saint Denis qui utilise une police avec une chasse très réduite.

#8

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

  • Assigné à changé de Nicolas Roche à Thomas Jund

Du coup tu valides Thomas ?

#9

Mis à jour par Thomas Jund il y a plus de 4 ans

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

yep

#10

Mis à jour par Thomas Jund il y a plus de 4 ans

  • Assigné à changé de Thomas Jund à Nicolas Roche
#11

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

Modification de dernière minute (pour #38878)
  • en fait e pas prendre les couleurs de la charte graphique
  • afficher un titre à côté du logo -> utiliser le "Titre général" modifiable via les options de l'intégration graphique dans l'IHM,
    (que je n'aurais pas du le rendre transparent mais juste décaler pour le faire disparaître)
#12

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

À nouveau :
  • stockage du logo dans les ressources (comme sur clapotis)
  • adaptation du header pour les mobiles (diminution de la hauteur et suppression du titre du site)
#13

Mis à jour par Thomas Noël il y a plus de 4 ans

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

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

  • Statut changé de Solution validée à Résolu (à déployer)
commit 3c5e5bc8a901707fd874cca9d5b22ec0f881eced
Author: Nicolas ROCHE <nroche@entrouvert.com>
Date:   Sat Dec 21 21:19:30 2019 +0100

    saint-denis: correct nav elements (#38681)
#15

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

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

Formats disponibles : Atom PDF