Projet

Général

Profil

Development #67614

developper un nouveau thème pour la Métropole Européenne de Lille

Ajouté par Serghei Mihai (congés, retour 15/05) il y a presque 2 ans. Mis à jour il y a plus d'un an.

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

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

Même si les maquettes ne sont pas définitivement validées et ça a l'air de partir encore pour un tour: #66579


Fichiers

0001-add-new-theme-for-M-tropole-Europ-enne-de-Lille-6761.patch (23,2 ko) 0001-add-new-theme-for-M-tropole-Europ-enne-de-Lille-6761.patch Serghei Mihai (congés, retour 15/05), 21 juillet 2022 16:39
accueil.png (221 ko) accueil.png Serghei Mihai (congés, retour 15/05), 21 juillet 2022 16:39
accueil-mobile.png (146 ko) accueil-mobile.png Serghei Mihai (congés, retour 15/05), 21 juillet 2022 16:39
formulaire-mobile.png (229 ko) formulaire-mobile.png Serghei Mihai (congés, retour 15/05), 21 juillet 2022 16:39
formulaire.png (624 ko) formulaire.png Serghei Mihai (congés, retour 15/05), 21 juillet 2022 16:39
0001-add-new-theme-for-M-tropole-Europ-enne-de-Lille-6761.patch (22,7 ko) 0001-add-new-theme-for-M-tropole-Europ-enne-de-Lille-6761.patch Serghei Mihai (congés, retour 15/05), 18 août 2022 15:32
0001-add-new-theme-for-M-tropole-Europ-enne-de-Lille-6761.patch (59,4 ko) 0001-add-new-theme-for-M-tropole-Europ-enne-de-Lille-6761.patch Serghei Mihai (congés, retour 15/05), 23 août 2022 15:37
mel1-mobile.png (48,2 ko) mel1-mobile.png Serghei Mihai (congés, retour 15/05), 23 août 2022 15:37
mel1.png (251 ko) mel1.png Serghei Mihai (congés, retour 15/05), 23 août 2022 15:37
0001-add-new-theme-for-M-tropole-Europ-enne-de-Lille-6761.patch (59,2 ko) 0001-add-new-theme-for-M-tropole-Europ-enne-de-Lille-6761.patch Serghei Mihai (congés, retour 15/05), 25 août 2022 17:40
0001-add-new-theme-for-M-tropole-Europ-enne-de-Lille-6761.patch (59,3 ko) 0001-add-new-theme-for-M-tropole-Europ-enne-de-Lille-6761.patch Serghei Mihai (congés, retour 15/05), 26 août 2022 09:21

Demandes liées

Lié à Intégrations graphiques Publik - Bug #68609: lille-metropole-2022: le fond du formulaire doit être blancFermé01 septembre 2022

Actions

Révisions associées

Révision c8d7c98d (diff)
Ajouté par Serghei Mihai (congés, retour 15/05) il y a plus d'un an

add new theme for Métropole Européenne de Lille (#67614)

Historique

#2

Mis à jour par Frédéric Péters il y a presque 2 ans

/* @media screen and ($max-mobile-viewport) { */
/*     div#nav, .site-header { */
/*         text-align: left; */
/*         position: static; */
/*         width: auto; */
/*         padding-right: 0; */
/*         border-bottom: 0; */
...

Si ça doit être supprimé, que ça soit supprimé.

~~

Ça n'aurait pas dû être proposé, puis ça n'aurait pas du passer notre validation, mais le texte blanc sur fond cyan (exemple le bouton "suivant"), il manque de contraste, 2,22:1, il faudrait 3:1.

#3

Mis à jour par Serghei Mihai (congés, retour 15/05) il y a presque 2 ans

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

Frédéric Péters a écrit :

Ça n'aurait pas dû être proposé, puis ça n'aurait pas du passer notre validation, mais le texte blanc sur fond cyan (exemple le bouton "suivant"), il manque de contraste, 2,22:1, il faudrait 3:1.

Tu as raison et je n'ai pas eu le réflexe de checker ça, faisant confiance à Jarticule.
Brice essaie de corriger le tir.

#4

Mis à jour par Serghei Mihai (congés, retour 15/05) il y a plus d'un an

Au final on fait sans le blue cyan: #66579#note-12

#5

Mis à jour par Serghei Mihai (congés, retour 15/05) il y a plus d'un an

Nouveau patch comprenant les quelques images fournies par J'articule.

#6

Mis à jour par Thomas Jund il y a plus d'un an

peut supprimer les extend implicites :

%button {
-    @extend %button;

%cell-links-list {
-    @extend %cell-links-list;

Links list
(avis perso) La couleur du hover des liens pose des problèmes de contraste


Toplinks
Pour supprimer le style "classic" des toplinks, tu peux maintenant utilisé la var `$toplinks-style: none;`
Ça t'évite quelques lignes de code CSS

#toplinks {
    top: 7px;
-    padding: 0;
    overflow: hidden;
    text-overflow: ellipsis;
-    box-shadow: none;
-    border: none;
-    border-radius: 0;
-    background: transparent;
-    padding-right: 0;
    right: 1rem;
    text-transform: uppercase;
    display: flex;
    align-items: center;

Main nav

  • z-index de la nav mobile risque d'être insuffisant dans certains cas. voir nav en mode $left-to-right par laquelle je suis monté à 2001 pour la btn et 2002 pour la ul.
  • C'est assez étrange d'avoir la nav fixed mais pas le bouton. Il suffit de scroller un peu pour voir la croix de fermeture disparaitre. Peut être troublant.

Formulaires

  • (avis perso) la bordure des hover des inputs en rouge. Ça fait input en erreur. Mauvaise idée.
  • (à essayer) ton cancel button est déjà codé dans le core, il correspond à %inverted-button :
+ $cancel-button-style: '%inverted-button';

// Parce que tu as surchargé %button, alors il faut hérité aussi de cette surcharge (un truc relou je trouve).
+ %inverted-button {
+    @extend %button;
+ }
// Ou alors plus élégant (de mon point de vue)
- %button {
+ %button, %inverted-button {
    text-transform: uppercase;
    …

- %cancel-button {
-    background: $primary-color;
-    border: 2px solid $primary-color;
-    color: white;
-    &:hover {
-        background: white;
-        color: $primary-color;
-    }
- }

#7

Mis à jour par Serghei Mihai (congés, retour 15/05) il y a plus d'un an

Thomas Jund a écrit :

peut supprimer les extend implicites :

[...]

Good catch, merci.

Links list
(avis perso) La couleur du hover des liens pose des problèmes de contraste

Je trouve aussi mais j'espère qu'ils reviendront dessus lorsque qu'ils testeront le thème en recette.

Toplinks
Pour supprimer le style "classic" des toplinks, tu peux maintenant utilisé la var `$toplinks-style: none;`

Fait, merci.

  • z-index de la nav mobile risque d'être insuffisant dans certains cas. voir nav en mode $left-to-right par laquelle je suis monté à 2001 pour la btn et 2002 pour la ul.
  • C'est assez étrange d'avoir la nav fixed mais pas le bouton. Il suffit de scroller un peu pour voir la croix de fermeture disparaitre. Peut être troublant.

Corrigé.

  • (avis perso) la bordure des hover des inputs en rouge. Ça fait input en erreur. Mauvaise idée.

Je les laisse le voir et faire un retour.

#9

Mis à jour par Thomas Jund il y a plus d'un an

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

OK pour moi.

#10

Mis à jour par Serghei Mihai (congés, retour 15/05) il y a plus d'un an

  • Statut changé de Solution validée à Résolu (à déployer)
commit c8d7c98d235304de8eb5039c397ae4b969362d73 (origin/main, origin/HEAD)
Author: Serghei Mihai <smihai@entrouvert.com>
Date:   Tue Jul 5 17:25:29 2022 +0200

    add new theme for Métropole Européenne de Lille (#67614)
#11

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

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

Mis à jour par Brice Mallet il y a plus d'un an

  • Lié à Bug #68609: lille-metropole-2022: le fond du formulaire doit être blanc ajouté
#13

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

Automatic expiration

Formats disponibles : Atom PDF