Development #67614
developper un nouveau thème pour la Métropole Européenne de Lille
0%
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
Demandes liées
Révisions associées
Historique
Mis à jour par Serghei Mihai (congés, retour 15/05) il y a presque 2 ans
- Fichier 0001-add-new-theme-for-M-tropole-Europ-enne-de-Lille-6761.patch 0001-add-new-theme-for-M-tropole-Europ-enne-de-Lille-6761.patch ajouté
- Fichier accueil.png accueil.png ajouté
- Fichier accueil-mobile.png accueil-mobile.png ajouté
- Fichier formulaire.png formulaire.png ajouté
- Fichier formulaire-mobile.png formulaire-mobile.png ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
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.
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.
Mis à jour par Serghei Mihai (congés, retour 15/05) il y a plus d'un an
- Fichier 0001-add-new-theme-for-M-tropole-Europ-enne-de-Lille-6761.patch 0001-add-new-theme-for-M-tropole-Europ-enne-de-Lille-6761.patch ajouté
- Statut changé de En cours à Solution proposée
Au final on fait sans le blue cyan: #66579#note-12
Mis à jour par Serghei Mihai (congés, retour 15/05) il y a plus d'un an
- Fichier 0001-add-new-theme-for-M-tropole-Europ-enne-de-Lille-6761.patch 0001-add-new-theme-for-M-tropole-Europ-enne-de-Lille-6761.patch ajouté
- Fichier mel1-mobile.png mel1-mobile.png ajouté
- Fichier mel1.png mel1.png ajouté
Nouveau patch comprenant les quelques images fournies par J'articule.
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; - } - }
Mis à jour par Serghei Mihai (congés, retour 15/05) il y a plus d'un an
- Fichier 0001-add-new-theme-for-M-tropole-Europ-enne-de-Lille-6761.patch 0001-add-new-theme-for-M-tropole-Europ-enne-de-Lille-6761.patch ajouté
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.
Mis à jour par Serghei Mihai (congés, retour 15/05) il y a plus d'un an
- Fichier 0001-add-new-theme-for-M-tropole-Europ-enne-de-Lille-6761.patch 0001-add-new-theme-for-M-tropole-Europ-enne-de-Lille-6761.patch ajouté
Nouveau patch corrigeant la position du bouton du menu burger.
Mis à jour par Thomas Jund il y a plus d'un an
- Statut changé de Solution proposée à Solution validée
OK pour moi.
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)
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 Brice Mallet il y a plus d'un an
- Lié à Bug #68609: lille-metropole-2022: le fond du formulaire doit être blanc ajouté
add new theme for Métropole Européenne de Lille (#67614)