Development #70027
nimes: ajustements intégration graphique
0%
Description
- Définir une barre latérale droite similaire à la barre latérale.
- Rendre cliquable l'image des listes de liens ayant une image définie et ne contenant qu'un seul lien.
- Passer le padding vertical des cellules de type liste de lien à 10px, uniquement en desktop.
- Utiliser un header rouge, des chevrons et retirer la bordure sur les listes de liens dans la barre latérale droite.
- Utiliser un header rouge, une plus grande marge et permettre de définir des icônes dans la liste de liens "Accès rapide".
- Définir trois classes pour les liens de la liste de liens accès rapide, pour les boutons Payer, Signaler et Prendre RDV.
- Ajouter une bordure rouge au champ texte de la cellule recherche.
- Remplacer le texte "Rechercher" du bouton de la cellule recherche par une icône de loupe.
- Réduire la hauteur de l'image de banière des pages à 150px.
- Cacher les boutons de connexion / inscription.
Fichiers
Demandes liées
Révisions associées
nimes: make picture clickable in single-link link list cells (#70027)
nimes: lower link-list-cells padding on desktop (#70027)
nimes: change style of link list in right sidebar (#70027)
nimes: add icon classes for links of link-list-cell (#70027)
nimes: change search button content, add border to search input (#70027)
nimes: set page image height to 150px (#70027)
nimes: hide toplinks (#70027)
Historique
Mis à jour par Corentin Séchet il y a plus d'un an
- Fichier 0002-nimes-make-picture-clickable-in-single-link-link-lis.patch 0002-nimes-make-picture-clickable-in-single-link-link-lis.patch ajouté
- Fichier 0007-nimes-add-icon-classes-for-links-of-link-list-cell.patch 0007-nimes-add-icon-classes-for-links-of-link-list-cell.patch ajouté
- Fichier 0010-nimes-hide-toplinks-70027.patch 0010-nimes-hide-toplinks-70027.patch ajouté
- Fichier 0004-nimes-change-style-of-link-list-in-right-sidebar-700.patch 0004-nimes-change-style-of-link-list-in-right-sidebar-700.patch ajouté
- Fichier 0006-nimes-fix-link-list-image-scaling-70027.patch 0006-nimes-fix-link-list-image-scaling-70027.patch ajouté
- Fichier 0003-nimes-lower-link-list-cells-padding-on-desktop-70027.patch 0003-nimes-lower-link-list-cells-padding-on-desktop-70027.patch ajouté
- Fichier 0001-nimes-add-a-two-sidebars-page-template-70027.patch 0001-nimes-add-a-two-sidebars-page-template-70027.patch ajouté
- Fichier 0005-nimes-fix-content-top-margin-70027.patch 0005-nimes-fix-content-top-margin-70027.patch ajouté
- Fichier 0008-nimes-change-search-button-content-add-border-to-sea.patch 0008-nimes-change-search-button-content-add-border-to-sea.patch ajouté
- Fichier 0009-nimes-set-page-image-height-to-150px-70027.patch 0009-nimes-set-page-image-height-to-150px-70027.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Corentin Séchet il y a plus d'un an
- Fichier export_page_nimes_20221011.json export_page_nimes_20221011.json ajouté
Mis à jour par A. Berriot il y a plus d'un an
- Statut changé de Solution proposée à Solution validée
Testé en local, tout me semble okay.
Je pense que le contenu du mixin pour les icones en 0007 gagnerait à être plus léger (et ne contenir que ce qui est vraiment variable, c'est à dire le nom de classe et l'URL de l'icone), histoire de ne pas générer énormément de CSS en doublon. Mais dans le cas présent, ça ne justifie pas de réécriture pour moi, il n'est appelé que 3 fois.
Mis à jour par Thomas Jund il y a plus d'un an
patch 01
Si la sidebar de droite doit avoir les mêmes carac que celle de gauche, autant les synchroniser en utilisant les vars de la sidebar. Et utiliser .central-content à la place de
.gru-content
.central-content div#right-sidebar { @media ($min-desktop-viewport) { flex: 0 0 $sidebar-width; max-width: $sidebar-max-width; } .cell { margin-right: $sidebar-columns-gutter; } }
Mis à jour par Thomas Jund il y a plus d'un an
Rendre cliquable l'image des listes de liens ayant une image définie et ne contenant qu'un seul lien.
Patch 02, demande étonnante, pourquoi ne pas plutôt utiliser une cellule lien ?
Patch 04, il est de bonne pratique d'indiquer le nom de l'icon utilisé
content: "\f054\f054"; // chevrons-right
Sinon, si jamais, il existe l'icon du double chevron : "angle-double-right"
Patch 06
L'image a déjà un max-width 100% il me semble. Si on te demande de forcer un width 100%, c'est dans le cas où tu as besoin d'agrandir une image, ce qui est une mauvaise idée d'un point du vue qualité visuelle.
Si par défaut la taille de crop de l'image est trop petit par rapport à la taille de la cellule, tu peux le modifier (cell_picture_size).
Sinon, c'est un problème éditorial : il doivent utiliser une image de meilleur qualité.
Mis à jour par Thomas Jund il y a plus d'un an
Patch 08
Ne pas utiliser `display: none` sur le label du bouton, il doit rester accessible. Utiliser `@extend .sr-only`
Et idem que pour patch 04 : ajouter le nom de l'icone.
Mis à jour par Corentin Séchet il y a plus d'un an
- Lié à Bug #69398: Le nom du site, lorqu'il est masqué, apparaît quand même sur iPhone ajouté
Mis à jour par Corentin Séchet il y a plus d'un an
- Lié à Bug #69398: Le nom du site, lorqu'il est masqué, apparaît quand même sur iPhone supprimé
Mis à jour par Thomas Jund il y a plus d'un an
Ne pas utiliser `display: none` sur le label du bouton
Oubli, y'a un aria-label dessus. Tout va bien comme ça.
Mis à jour par Corentin Séchet il y a plus d'un an
- Statut changé de Solution validée à Résolu (à déployer)
commit fb8b8a1f45fa647621e01a06a8a06ff389554a13 Author: Corentin Séchet <csechet@entrouvert.com> Date: Mon Oct 10 21:49:33 2022 +0200 nimes: hide toplinks (#70027) commit 0b770f8703fa2f507e7b322356a57e858f17701a Author: Corentin Séchet <csechet@entrouvert.com> Date: Mon Oct 10 21:49:18 2022 +0200 nimes: set page image height to 150px (#70027) commit ef0666e67983d32d37a8a540c012977a21fd97ca Author: Corentin Séchet <csechet@entrouvert.com> Date: Mon Oct 10 21:38:59 2022 +0200 nimes: change search button content, add border to search input (#70027) commit 93e99a048a87da8af9a434dec18e9620344f0fef Author: Corentin Séchet <csechet@entrouvert.com> Date: Mon Oct 10 18:18:16 2022 +0200 nimes: add icon classes for links of link-list-cell (#70027) commit 9532d85b9dd126e03cc1a8066fafbcaabe0d00e1 Author: Corentin Séchet <csechet@entrouvert.com> Date: Mon Oct 10 17:29:47 2022 +0200 nimes: change style of link list in right sidebar (#70027) commit 98370f87fc95d9ba768753a9a348d0323917bc4b Author: Corentin Séchet <csechet@entrouvert.com> Date: Mon Oct 10 15:43:52 2022 +0200 nimes: lower link-list-cells padding on desktop (#70027) commit 1a9141142600df09b0227b52cd9093e1d15e2874 Author: Corentin Séchet <csechet@entrouvert.com> Date: Mon Oct 10 15:41:08 2022 +0200 nimes: make picture clickable in single-link link list cells (#70027) commit efff24558fe7284d8bc7e6132cd4fc89bbacaa0a Author: Corentin Séchet <csechet@entrouvert.com> Date: Mon Oct 10 12:35:28 2022 +0200 nimes: add a two-sidebars page template (#70027)
Mis à jour par Corentin Séchet il y a plus d'un an
Merci Thomas, j'ai intégré tout ça et supprimé le patch qui set une width sur les images de listes de lien.
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 Corentin Séchet il y a plus d'un an
Thomas Jund a écrit :
Patch 02, demande étonnante, pourquoi ne pas plutôt utiliser une cellule lien ?
Pardon, j'ai oublié ce point : c'est suite à la demande d’Anaïs, qui m'a dit qu'il y avait des discussions en cours pour déprécier la cellule lien au profit de la cellule liste de liens, qui fait double emploi.
Mis à jour par Lauréline Guérin il y a 12 mois
- Lié à Development #77660: Nïmes: supprimer le display none sur #toplinks ajouté
nimes: add a two-sidebars page template (#70027)