Projet

Général

Profil

Development #67231

Nouveau thème : Le Plessis-Trévise

Ajouté par Paul Marillonnet il y a presque 2 ans. Mis à jour il y a plus d'un an.

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

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

Intégration native, sur la base du institutionnel : https://www.leplessistrevise.fr/


Fichiers

0001-leplessistrevise-add-theme-67231.patch (43,3 ko) 0001-leplessistrevise-add-theme-67231.patch Paul Marillonnet, 17 août 2022 18:01
desktop-3.png (745 ko) desktop-3.png Paul Marillonnet, 17 août 2022 18:02
mobile-3.png (1,56 Mo) mobile-3.png Paul Marillonnet, 17 août 2022 18:02
mobile-2.png (118 ko) mobile-2.png Paul Marillonnet, 17 août 2022 18:02
desktop-2.png (182 ko) desktop-2.png Paul Marillonnet, 17 août 2022 18:02
mobile-1.png (409 ko) mobile-1.png Paul Marillonnet, 17 août 2022 18:02
desktop-1.png (488 ko) desktop-1.png Paul Marillonnet, 17 août 2022 18:02
entete-mobile.png (30,6 ko) entete-mobile.png Chloé Girard, 23 août 2022 14:21
Screenshot_20220905_131729.png (60,4 ko) Screenshot_20220905_131729.png Titres taille et padding A. Berriot, 05 septembre 2022 13:42
Screenshot_20220905_130716.png (13,7 ko) Screenshot_20220905_130716.png Boutons A. Berriot, 05 septembre 2022 13:42
Screenshot 2022-09-05 at 13-28-43 Portail - Gestion du compte.png (78,5 ko) Screenshot 2022-09-05 at 13-28-43 Portail - Gestion du compte.png Largeur header A. Berriot, 05 septembre 2022 13:42
Screenshot_20220905_131608.png (117 ko) Screenshot_20220905_131608.png Titres cellules A. Berriot, 05 septembre 2022 13:42
0001-leplessistrevise-add-theme-67231.patch (44 ko) 0001-leplessistrevise-add-theme-67231.patch Paul Marillonnet, 06 septembre 2022 14:21
mobile3.png (373 ko) mobile3.png Paul Marillonnet, 06 septembre 2022 14:21
desktop3.png (601 ko) desktop3.png Paul Marillonnet, 06 septembre 2022 14:21
mobile2.png (1,71 Mo) mobile2.png Paul Marillonnet, 06 septembre 2022 14:21
desktop2.png (795 ko) desktop2.png Paul Marillonnet, 06 septembre 2022 14:21
mobile1.png (92,2 ko) mobile1.png Paul Marillonnet, 06 septembre 2022 14:21
desktop1.png (159 ko) desktop1.png Paul Marillonnet, 06 septembre 2022 14:21
header-desktop.png (51,2 ko) header-desktop.png Paul Marillonnet, 07 septembre 2022 16:23
0001-leplessistrevise-add-theme-67231.patch (43,8 ko) 0001-leplessistrevise-add-theme-67231.patch Paul Marillonnet, 07 septembre 2022 16:23
header-mobile.png (40,2 ko) header-mobile.png Paul Marillonnet, 07 septembre 2022 16:23

Révisions associées

Révision 99f5b0a3 (diff)
Ajouté par Paul Marillonnet il y a plus d'un an

leplessistrevise: add theme (#67231)

Historique

#2

Mis à jour par Paul Marillonnet il y a plus d'un an

Quelques captures pour une idée de ce que ça à quoi ça pourrait ressembler, avec notamment l’ajout du gabarit combo trois colonnes, car structuration récurrente sur le site institutionnel.

#3

Mis à jour par Paul Marillonnet il y a plus d'un an

(Et, bien sûr, branche basée sur #68144.)

#4

Mis à jour par Chloé Girard il y a plus d'un an

Qui d'autre peut porter un regard avisé sur cette proposition de Paul avec moi ?

Mes remarques :
  • très ok de manière générale.
Sur les en-têtes (bandeaux) :
  • manque en en-tête un titre du genre "Démarches en ligne"
  • pour la proposition "mobile", la différence de bordures g/d entre l'en-tête et le corps de la page ne va pas : captures #68676, 68678 et 68675
  • Question en-tête mobile, si je regarde des sites comme https://vosdemarches.cannes.com/ ou demarches.cantal.fr on a des conflits entre le logo, les boutons de connexion, le titre, etc. Le site https://portail-angouleme.test.entrouvert.org/ est plutôt pas mal avec le menu burger à droite, le logo sous les boutons de connexion, et un éventuel titre "Démarches en ligne". On peut caser tout ça ? Je mets en pj un exemple bidouillé par mes soins, qui ne vaut, grosso modo que pour les proportions.
  • Question en-tête desktop, je garderais le "Ville de" au-dessus de Le Plessis-Trévise. Je garderais autour du logo les mêmes agencements du texte, en remplaçant "Toute l'info de votre ville au quotidien" par "Démarches en ligne".
Version desktop :
  • le version page d'accueil sur 3 colonnes me plaît. Intégrer la recherche et le code de suivi à ces 3 colonnes ?

Pour les deux versions : a priori il n'y aura pas besoin de menus déroulants mais juste trois liens dans le menu du haut : Accueil, Mes demandes, Mon compte.

D'autres choses que je n'aurais pas vues ?

#5

Mis à jour par Paul Marillonnet il y a plus d'un an

Merci pour tes retours Chloé, je vais intégrer tout cela. Le patche est un peu brut de fonderie, mais le but était de proposer quelque chose pour une première relecture que je prendrai en compte à mon retour de congés. Donc si un·e dév intégrateur·rice veut relire, most welcome.

#6

Mis à jour par Thomas Jund (congés, retour le 29/04) il y a plus d'un an

Pour la taille du logo et l'affichage du titre du site, tu peux maintenant regarder du côté des variables
$header-logo-size
$show-site-title

Tu peux gagner quelques lignes de code custom.
Ticket avec exemple #64893

#7

Mis à jour par A. Berriot il y a plus d'un an

En complément des retours de Chloé et Thomas, je fais une relecture plus complète (avec test en local).

Je n'ai rien vu de bloquant, ce sont surtout des petites choses / différences avec le site https://www.leplessistrevise.fr/ :

- Le line-height semble différent (un peu moins compact sur leur site), il est à 1.5em chez eux et 1.4 dans le thème. Ça vaut peut être le coup d'utiliser le tout nouveau https://dev.entrouvert.org/issues/68540 pour uniformiser
- Le style des boutons diffère (couleur, radius), je pense qu'on pourrait garder ce qui est présenté sur https://www.leplessistrevise.fr/ma-ville/autres/offres-demplois/ par exemple (cf pièce jointe)
- le thème applique un border radius assez important sur les input, que je ne retrouve pas sur https://www.leplessistrevise.fr/nous-contacter/envoyer-une-candidature/ par exemple. La couleur de focus est aussi différente (bleu sur leur site, pas de changement dans le thème)
- les titres 1 et 2 manquent parfois de padding, ce qui les colle à la bordure du contenant (cf pièce jointe)
- les titres de cellules sont peut-être un peu gros (cf pièce jointes). Je ne trouve pas d'équivalent à leurs cellules chez nous, mais sur des layouts à deux ou trois colonnes, avec les bordures, ça me parait trop chargé
- je constate une différence de largeur entre le header/nav et le main, cf pièce jointe. À mon sens, le header devrait faire la même taille que le main
- le width a 60% pose problème sur des écrans un peu étroits, cela fait beaucoup d'espace perdu. Je pense qu'on pourrait mettre un max width à 840 (pour matcher la taille d'une colonne principale de leur site, par exemple celle de https://www.leplessistrevise.fr/temps-libre/culture/mediatheque-jacques-duhamel/) et simplement passer à width = 100% sur les écrans plus étroits

#8

Mis à jour par Paul Marillonnet il y a plus d'un an

Je reprends point par point les retours faits ici :

–––––

  • manque en en-tête un titre du genre "Démarches en ligne"

Ok, j’ai ajouté l’html nécessaire pour l’entête dans le gabarit trois colonnes.

  • pour la proposition "mobile", la différence de bordures g/d entre l'en-tête et le corps de la page ne va pas : captures #68676, 68678 et 68675

Pas compris, désolé, je ne vois pas de quoi tu parles sur les captures :/

  • Question en-tête mobile, si je regarde des sites comme https://vosdemarches.cannes.com/ ou demarches.cantal.fr on a des conflits entre le logo, les boutons de connexion, le titre, etc. Le site https://portail-angouleme.test.entrouvert.org/ est plutôt pas mal avec le menu burger à droite, le logo sous les boutons de connexion, et un éventuel titre "Démarches en ligne". On peut caser tout ça ? Je mets en pj un exemple bidouillé par mes soins, qui ne vaut, grosso modo que pour les proportions.

Sur le portail d’Angoulême moi j’ai le menu burger à gauche (?)
Par contre oui, le menu responsive qui s’ouvre de gauche à droite c’est plus propre et ça colle davantage au site institutionnel. C’est natif dans base-theme et configuré dans cette nouvelle version de ce patche.
Pour le sous titre du site, à ma connaissance c’est pas natif par contre. Je ne me sens pas trop de modifier le gabarit là-dessus, j’ai peur que ça casse des trucs :/

  • Question en-tête desktop, je garderais le "Ville de" au-dessus de Le Plessis-Trévise. Je garderais autour du logo les mêmes agencements du texte, en remplaçant "Toute l'info de votre ville au quotidien" par "Démarches en ligne".

Même remarque que ci-dessus : pas natif sauf erreur de ma part.

Version desktop :
  • le version page d'accueil sur 3 colonnes me plaît. Intégrer la recherche et le code de suivi à ces 3 colonnes ?

Oui, le gabarit est modifié dans cette nouvelle version du patch pour y inclure un espace d’entête sur cette page à 3 colonnes.

Pour les deux versions : a priori il n'y aura pas besoin de menus déroulants mais juste trois liens dans le menu du haut : Accueil, Mes demandes, Mon compte.

Ok, je laisse quand même en l’état au cas où les sous-menu s’avèrent nécessaires par la suite.

–––––

Pour la taille du logo et l'affichage du titre du site, tu peux maintenant regarder du côté des variables
$header-logo-size
$show-site-title

Tu peux gagner quelques lignes de code custom.
Ticket avec exemple #64893

Top, merci, je ne connaissais pas ces deux variables, que j’ai incluses dans cette nouvelle version du patche.

––––––

- Le line-height semble différent (un peu moins compact sur leur site), il est à 1.5em chez eux et 1.4 dans le thème. Ça vaut peut être le coup d'utiliser le tout nouveau https://dev.entrouvert.org/issues/68540 pour uniformiser

Bien vu, c’est corrigé, avec la nouvelle variable que tu mentionnes ici.

- Le style des boutons diffère (couleur, radius), je pense qu'on pourrait garder ce qui est présenté sur https://www.leplessistrevise.fr/ma-ville/autres/offres-demplois/ par exemple (cf pièce jointe)

Corrigé aussi.

- le thème applique un border radius assez important sur les input, que je ne retrouve pas sur https://www.leplessistrevise.fr/nous-contacter/envoyer-une-candidature/ par exemple. La couleur de focus est aussi différente (bleu sur leur site, pas de changement dans le thème)

Oui, craquage de mon côté, c’est pas du tout présent sur le site institutionnel. J’ai corrigé.

- les titres 1 et 2 manquent parfois de padding, ce qui les colle à la bordure du contenant (cf pièce jointe)

Un style spécifique aux titres, sans bordure ni fond pour la cellule parente, à l’aide de la classe css custom page-title sur la cellule texte posée dans la page combo (classe stylée dans cette nouvelle version du patche).

- les titres de cellules sont peut-être un peu gros (cf pièce jointes). Je ne trouve pas d'équivalent à leurs cellules chez nous, mais sur des layouts à deux ou trois colonnes, avec les bordures, ça me parait trop chargé

Ok, je suis passé de 30px à 20px, ça a l’air mieux, en effet.

- je constate une différence de largeur entre le header/nav et le main, cf pièce jointe. À mon sens, le header devrait faire la même taille que le main

Ok, ça ne me choquait pas en l’état mais je n’ai pas l’œil pour ce genre de choses. Je modifie pour uniformiser la largeur.

- le width a 60% pose problème sur des écrans un peu étroits, cela fait beaucoup d'espace perdu. Je pense qu'on pourrait mettre un max width à 840 (pour matcher la taille d'une colonne principale de leur site, par exemple celle de https://www.leplessistrevise.fr/temps-libre/culture/mediatheque-jacques-duhamel/) et simplement passer à width = 100% sur les écrans plus étroits

Oui tu as raison, c’est corrigé.

––––––

Nouvelle version du patche et les captures à jour qui vont avec.

#9

Mis à jour par A. Berriot il y a plus d'un an

Retesté à l'instant en local, les points que tu as adressé dans ton patche me semblent nickel.

Il y a juste $font-line-height ne semble pas fonctionner (line-height toujours à 1.4 chez moi) mais c'est possible que le problème ne se pose que sur ma machine, car sur le thème MNHN sur lequel je travaille depuis ce matin elle ne marche pas non plus.

Par contre oui, le menu responsive qui s’ouvre de gauche à droite c’est plus propre et ça colle davantage au site institutionnel. C’est natif dans base-theme et configuré dans cette nouvelle version de ce patche.

Je découvre et ça va me servir, merci !

Tu peux considérer que le ticket est validé pour moi, je ne change pas le statut pour que Chloé puisse répondre à tes questions.

#10

Mis à jour par Paul Marillonnet il y a plus d'un an

Agate Berriot a écrit :

Il y a juste $font-line-height ne semble pas fonctionner (line-height toujours à 1.4 chez moi) mais c'est possible que le problème ne se pose que sur ma machine, car sur le thème MNHN sur lequel je travaille depuis ce matin elle ne marche pas non plus.

Oui je crois que c’est du côté #68540 qu’il manque quelque chose. De mon côté ça ne fonctionne pas non plus. Même en définissant cette variable c’est la valeur par défaut dans static/includes/_general.scss qui est attrapée.

Tu peux considérer que le ticket est validé pour moi, je ne change pas le statut pour que Chloé puisse répondre à tes questions.

Top, merci.

#11

Mis à jour par Paul Marillonnet il y a plus d'un an

  • Statut changé de Solution proposée à En cours

Discussion de vive-voix avec Chloé sur ses retours que je ne comprenais pas par écrit. La seule chose qui reste est le titre dans l’entête qu’elle trouve affiché de façon trop austère par rapport au site institutionnel.
Je vais voir dans quelle mesure je peux adapter l’affichage du texte (ou carrément prendre une image à la place, et remettre le texte aligné à -10000px).

#12

Mis à jour par Paul Marillonnet il y a plus d'un an

Paul Marillonnet a écrit :

Je vais voir dans quelle mesure je peux adapter l’affichage du texte (ou carrément prendre une image à la place, et remettre le texte aligné à -10000px).

Allez, plutôt que d’inventer un truc plus ou moins (surtout moins) compatible base-theme natif, je prends une image.

#13

Mis à jour par Chloé Girard il y a plus d'un an

Je tamponne, je signe, je re-tamponne. Go. C'est parfait.

#14

Mis à jour par Paul Marillonnet il y a plus d'un an

Ack d’Agate et de Chloé, on considère ça validé. Je vais pousser tagguer déployer.

#15

Mis à jour par Paul Marillonnet il y a plus d'un an

  • Statut changé de Solution proposée à Résolu (à déployer)
commit 99f5b0a32e9ed2d92dfe6f98815cdf67ae86c8cd
Author: Paul Marillonnet <pmarillonnet@entrouvert.com>
Date:   Fri Aug 12 10:48:11 2022 +0200

    leplessistrevise: add theme (#67231)
#16

Mis à jour par Transition automatique il y a plus d'un an

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

Mis à jour par Chloé Girard il y a plus d'un an

Euh, désolée Paul mais je ne sais pas où sont les ressources img ? Je les récupère comment ? Où ?

#18

Mis à jour par Paul Marillonnet il y a plus d'un an

Chloé Girard a écrit :

Euh, désolée Paul mais je ne sais pas où sont les ressources img ? Je les récupère comment ? Où ?

(Vu en direct avec Chloé, j’ai déposé les ressources nécessaires sur l’instance de recette.)

#19

Mis à jour par Transition automatique il y a plus d'un an

Automatic expiration

Formats disponibles : Atom PDF