Development #70365
leplessistrevise : corrections thème
0%
Description
Intégration des remarques de l’équipe projet, visibles ici : https://dev.entrouvert.org/attachments/70143
Fichiers
Révisions associées
leplessistrevise: bring minor fix to theme, 2 (#70365)
Historique
Mis à jour par Paul Marillonnet il y a plus d'un an
- Statut changé de Nouveau à En cours
- Assigné à mis à Paul Marillonnet
Mis à jour par Paul Marillonnet il y a plus d'un an
- Fichier 0001-leplessistrevise-bring-minor-fixes-to-theme-70365.patch 0001-leplessistrevise-bring-minor-fixes-to-theme-70365.patch ajouté
- Statut changé de En cours à Solution proposée
- Patch proposed changé de Non à Oui
Une somme de tous petits trucs à corriger, tel qu’apparaissant dans le pdf de la description de ce ticket.
Giga flemme de tout découper en plusieurs tickets, par contre je peux découper en plusieurs patches si la relecteurice me le demande.
Mis à jour par Frédéric Péters il y a plus d'un an
Pour les modifs sur le bouton hamburger, pour la couleur des barres, tu as $nav-button-color, plutôt que aller viser
.icon-bar { background-color: white; }
(je n'ai pas regardé super attentivement, je pars de l'idée que le rendu correspond à ce qu'ils demandent)
Mis à jour par Paul Marillonnet il y a plus d'un an
Frédéric Péters a écrit :
Pour les modifs sur le bouton hamburger, pour la couleur des barres, tu as $nav-button-color, plutôt que aller viser
[...]
(je n'ai pas regardé super attentivement, je pars de l'idée que le rendu correspond à ce qu'ils demandent)
En fait pas si simple parce que le menu left-to-right présente un cas particulier (dans static/includes/_nav.scss) :
div.gru-nav .gru-nav-button.toggled {
@if $responsive-menu == left-to-right {
background: transparent;
}
& .icon-bar {
@if $responsive-menu == left-to-right {
background-color: $nav-border-color;
}
}
// […]
}
On peut ajouter $nav-border-color dans le variable, mais alors il faut retirer la bordure à gauche de 4 pixels dont je ne comprends pas l’utilité :
@media screen and (max-width: $mobile-limit) {
body {
border-left: 0px;
}
}
Billard à trois bandes, pas sûr qu’on veuille ça.
Mis à jour par Frédéric Péters il y a plus d'un an
- Statut changé de Solution proposée à Solution validée
Ok non ne plongeons pas là-dedans.
et en me basant toujours sur l'idée que le rendu correspond à tout ce qu'ils demandent dans leur pdf, je valide.
Mis à jour par A. Berriot il y a plus d'un an
Retours basés sur mes tests en local en déroulant leur pdf. Les corrections que tu as appliquées fonctionnent correctement chez moi pour tout ce qui n'est pas listé ci dessous :)
1.Zone de Recherche
Bouton « Recherche » :
– Supprimer l’ombre portée (y compris en
mode :hover)
L'ombre est toujours présente
2. Zone de suivi d’une démarche
Bouton « Valider » :– Supprimer l’ombre portée (y compris
en mode :hover)
Même constat ici ;)
Modifier l’expéditeur pour mettre « Ville du Plessis-Trévise »
Je ne vois rien dans le patch lié à ça, mais je ne suis pas sûre que ça doive se régler côté publik-base-theme, je pose juste ça là pour vérifier qu'on oublie pas quelque chose.
Autrement, c'est top, et tu peux considérer que c'est ok pour moi de merger quand les points que j'ai évoqués sont clarifiés / corrigés !
Mis à jour par Paul Marillonnet il y a plus d'un an
Agate Berriot a écrit :
L'ombre est toujours présente
[…]
Même constat ici ;)
Bien vu, corrigé, merci.
Modifier l’expéditeur pour mettre « Ville du Plessis-Trévise »
Je ne vois rien dans le patch lié à ça, mais je ne suis pas sûre que ça doive se régler côté publik-base-theme, je pose juste ça là pour vérifier qu'on oublie pas quelque chose.
Oui c’est la conf email hobo, j’ai modifié directement sur la recette.
Mis à jour par Paul Marillonnet il y a plus d'un an
- Statut changé de Solution validée à Résolu (à déployer)
commit 8d480659dc2a3ebf6107123ba1dae74d16a8da7d Author: Paul Marillonnet <pmarillonnet@entrouvert.com> Date: Mon Oct 17 17:12:49 2022 +0200 leplessistrevise: bring minor fixes to theme (#70365)
Mis à jour par Transition automatique il y a plus d'un an
- Statut changé de Résolu (à déployer) à Solution déployée
leplessistrevise: bring minor fixes to theme (#70365)