armentieres: fix respect guide de style & ajustements (#70745) #65
aberriot
Loading…
Reference in New Issue
No description provided.
Delete Branch "wip/70745-armentieres-ajustements"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Il y a encore des choses en WIP (des textes placeholder par endroit), j'attends les retours du client mais si on pouvait déjà envoyer ça ça leur permettrait une première phase de recettage. Il reste un aller / retour à faire, je fixerai les derniers points à ce moment là.
Bon, j'ai fait une relecture commit par commit avec tests en local. J'ai peu de commentaires à faire sur le code, il y a tellement de choses, je vais me concentrer sur le rendu.
Déjà en me basant sur https://gitea.entrouvert.org/csechet/sranko/src/branch/main/themes/armentieres pour la page d'accueil et les assets, je trouve que le rendu se rapproche en effet beaucoup plus de leurs demandes.
Voici les petites choses que j'ai remarquées, commentées vers le code qui matche possiblement.
@ -0,0 +241,4 @@}// Search cell.gru-content div.searchcell {On a un fond blanc pour le champ de recherche en barre latérale, alors qu'il est grisé dans le style guide :
Fixé, j'ai aussi retiré la box-shadow, aligné correctement l'icone et retiré la bordure.
@ -0,0 +4,4 @@position: relative;}.back-top {Ce bouton fonctionne très bien sur mobile, mais je le trouve pénible sur desktop, il se positionne par dessus le contenu chez moi, ce qui est pénible sachant qu'il y a de la place à droite. Dans certains cas, ça masque du contenu ou de sliens utiles (genre la croix pour enlever un fichier d'un formulaire)
Le client a demandé ça, je propose de les laisser constater que c'est pas pratique ou de proposer autre chose :)
Ça me va !
@ -0,0 +161,4 @@}}div#toplinks {Sur mobile, ça manque de padding en dessous des infos du compte, je dirai :
Fixé
@ -0,0 +35,4 @@// Tracking codediv#tracking-code {padding: 1rem;J'ai l'impression qu'il y a un souci de padding ici, chez moi le padding latéral et vértical ne sont pas identiques :
C'est peut-etre une vieille version de la page que tu as importé : il faut utiliser la disposition "Une colonne" et non "Manuel" pour le slot (en cliquant sur "Options" en haut à droite du slot contenu dans le back office de Combo), pour que les cellules soient alignées.
En disposition une colonne, j'ai ça :
Le souci de padding me semble oujours être là (c'est plus vide sur les côtés du widget de suivi)
J'ai mis le meme padding pour les deux cellules. Elles sont prévues pour s'afficher cote à cote ratio 1/2 1/2.
@ -0,0 +38,4 @@padding: 1rem;background: right / cover url('/assets/tracking-code:background');border-radius: $border-radius;.tracking-code-part {Toujours dans cette cellule, sur mobile à certaines résolutions, les bouton et l'input n'ont pas la même hauteur :
(aussi l'impression qu'il y a une marge ou un padding superflu dans le bouton signaler)
Sur mobile, j'ai fait en sorte que le bouton soit systématiquement en dessous du champ, comme demandé sur la maquette.
@ -0,0 +73,4 @@// Widgets%button {box-shadow: none;height: 35px;Les boutons m'ont l'air de manquer un petit de padding en haut et en bas par rapport à leur style guide
Style guide :
Je vais regarder, c'est peut etre un souci de taille de police, les boutons doivent faire 35px de haut (de mémoire).
J'ai passé la taille des boutons à 45px de haut. Je pense que c'est ce qui est fait sur le guide de style, sauf sur la cellule code de suivi et "signaler", où c'est une hauteur de 35px qui est voulue par le client (parce que pourquoi pas).
Pas pu retester en local, mais tout me semble bon à la relecture des derniers commits !
4f0f6f884dtofacba91462