Development #67231
Nouveau thème : Le Plessis-Trévise
0%
Fichiers
Révisions associées
Historique
Mis à jour par Paul Marillonnet il y a plus d'un an
- Fichier 0001-leplessistrevise-add-theme-67231.patch 0001-leplessistrevise-add-theme-67231.patch ajouté
- Fichier mobile-3.png mobile-3.png ajouté
- Fichier desktop-3.png desktop-3.png ajouté
- Fichier mobile-2.png mobile-2.png ajouté
- Fichier desktop-2.png desktop-2.png ajouté
- Fichier mobile-1.png mobile-1.png ajouté
- Fichier desktop-1.png desktop-1.png ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
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.
Mis à jour par Chloé Girard il y a plus d'un an
- Fichier entete-mobile.png entete-mobile.png ajouté
- Patch proposed changé de Oui à Non
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.
- 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".
- 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 ?
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.
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
Mis à jour par A. Berriot il y a plus d'un an
- Fichier Screenshot_20220905_131729.png Screenshot_20220905_131729.png ajouté
- Fichier Screenshot_20220905_130716.png Screenshot_20220905_130716.png ajouté
- Fichier Screenshot 2022-09-05 at 13-28-43 Portail - Gestion du compte.png Screenshot 2022-09-05 at 13-28-43 Portail - Gestion du compte.png ajouté
- Fichier Screenshot_20220905_131608.png Screenshot_20220905_131608.png ajouté
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
Mis à jour par Paul Marillonnet il y a plus d'un an
- Fichier 0001-leplessistrevise-add-theme-67231.patch 0001-leplessistrevise-add-theme-67231.patch ajouté
- Fichier mobile3.png mobile3.png ajouté
- Fichier desktop3.png desktop3.png ajouté
- Fichier mobile2.png mobile2.png ajouté
- Fichier desktop2.png desktop2.png ajouté
- Fichier mobile1.png mobile1.png ajouté
- Fichier desktop1.png desktop1.png ajouté
- Patch proposed changé de Non à Oui
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-titleTu 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.
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.
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.
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).
Mis à jour par Paul Marillonnet il y a plus d'un an
- Fichier 0001-leplessistrevise-add-theme-67231.patch 0001-leplessistrevise-add-theme-67231.patch ajouté
- Fichier header-desktop.png header-desktop.png ajouté
- Fichier header-mobile.png header-mobile.png ajouté
- Statut changé de En cours à Solution proposée
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.
Mis à jour par Chloé Girard il y a plus d'un an
Je tamponne, je signe, je re-tamponne. Go. C'est parfait.
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.
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)
Mis à jour par Transition automatique il y a plus d'un an
- Statut changé de Résolu (à déployer) à Solution déployée
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ù ?
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.)
leplessistrevise: add theme (#67231)