Development #63340
Créer le thème pour la Vendée
0%
Description
Sur base de https://edap.vendee.fr/ mais en utilisant le rouge #ED1B24 à la place du orange
Fichiers
Demandes liées
Révisions associées
Historique
Mis à jour par Brice Mallet il y a environ 2 ans
- Description mis à jour (diff)
- Statut changé de Nouveau à En cours
Mis à jour par Corentin Séchet il y a environ 2 ans
- Fichier 0001-vendee-cd85-create-theme-63340.patch 0001-vendee-cd85-create-theme-63340.patch ajouté
- Fichier combo_desktop.png combo_desktop.png ajouté
- Fichier combo_mobile.png combo_mobile.png ajouté
- Fichier wcs_desktop.png wcs_desktop.png ajouté
- Fichier wcs_mobile.png wcs_mobile.png ajouté
- Statut changé de En cours à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Thomas Jund (congés, retour le 29/04) il y a environ 2 ans
Je laisserai respirer à peine plus le titre les cellules :
$title-padding: 0.4rem 1rem 0.33rem 1rem;
Aucune raison de ne pas activer le mode PWA sur tous les thèmes, surtout que tu ne paramètres pas de favicon, autant laisser la possibilité de le charger depuis combo grace à l'inferface PWA.
"pwa_display": "standalone"
#toplinks n'est pas full width en desktop. Petite barre gris clair à droite.
#toplinks { right: 0; }
Le ":input" des widgets en rouge, ça fait champ en erreur. je choisirai autre chose.
Et le rouge sang "logo vendée" de manière global, je ne trouve ça pas terrible, ni sur les items de list ni sur les boutons. À utiliser par petite touches je dirais, là c'est globuleux. Mais j'ai rien à proposer de précis, alors si tu veux balancer comme ça, les vendéeens ont p-e l'habitude de voir cette couleur partout.
Tu multiplies un peu trop les bordures/filets je trouve (cells, input, buttons, nav)
Je conseil de tenter un `$cell-border: none;` pour alléger un peu.
Si tu souhaites ajouter le box-shadow à toutes les "cellules", même les block authentic, tu peux simpleemnt utiliser le selecteur sass `%cell` :
%cell { box-shadow: 3px 3px 10px $gray; }
Mis à jour par Corentin Séchet il y a presque 2 ans
- Fichier 0001-misc-add-title-field-to-text-cells-42968.patch ajouté
J'ai mis les boutons en noir, avec changement de couleur, pas de background sur le hover en rouge, et mis les liens en rouge uniquement sur le hover, c'est effectivement plus doux.
Mis à jour par Corentin Séchet il y a presque 2 ans
- Fichier
0001-misc-add-title-field-to-text-cells-42968.patchsupprimé
Mis à jour par Corentin Séchet il y a presque 2 ans
Mis à jour par Thomas Jund (congés, retour le 29/04) il y a presque 2 ans
- Statut changé de Solution proposée à Solution validée
Dernières remarques,
Je trouve que les inputs de formulaire ne sont pas très visibles (manque de contraste).
Peut-être ajouter une border par défaut qui les rendraient plus visibles
$widget-border: 2px solid $light-gray;
et réduire le contraste des boutons
$button-color: $dark-gray; $button-border: 2px solid $dark-gray; $widget-background: $x-light-gray;
As you want, je valide.
Mis à jour par Corentin Séchet il y a presque 2 ans
Merci beaucoup Thomas. Comme ils attendent ça en recette pour faire les retours et que je suis pas en avance, j'ouvre des tickets pour tes remarques et j'envoie celui-ci en recette.
Mis à jour par Corentin Séchet il y a presque 2 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 8a84aa9fdc5ae2953caf36ebfd490d0c853b8406 Author: Corentin Séchet <csechet@entrouvert.com> Date: Wed Mar 30 10:32:43 2022 +0200 vendee-cd85: create theme (#63340)
Mis à jour par Corentin Séchet il y a presque 2 ans
- Lié à Development #64786: vendee-cd85: Améliorer le contraste des boutons & inputs ajouté
Mis à jour par Transition automatique il y a presque 2 ans
- Statut changé de Résolu (à déployer) à Solution déployée
vendee-cd85: create theme (#63340)