entrouvert
/
publik-base-theme
15
1
Fork 1
Code Issues Pull Requests 8 Releases Wiki Activity

univ-nantes: theme tweaks to match client request (#71216) #10

Merged
aberriot merged 1 commits from wip/71216-univ-nantes-retours into main 2022-11-16 10:02:49 +01:00
Conversation 5 Commits 1 Files Changed 3 +94 -54
aberriot commented 2022-11-10 11:15:27 +01:00
Owner
Copy Link

Sur la page wcs (https://demarches-univ-nantes.test.entrouvert.org/formulaire-pour-tester-le-rendu-ne-pas-supprimer-ni-modifier/?cancelurl=https%3A//portail-univ-nantes.test.entrouvert.org/page-2-colonnes-1-barre-laterale/) :

  • pas assez large, on est sur une largeur de 800px, il faut l'augmenter (pas la barre latérale, qui est assez large, mais le contenu)

Sur une page combo (https://portail-univ-nantes.test.entrouvert.org/page-2-colonnes-1-barre-laterale/) :

  • ajouter un peu de largeur sur le contenu (barre latérale ok)
  • la cellule "lien vers une démarche" manque de padding (cellule dans la colonne de droite, en haut)
  • dans le footer, il faudrait avoir un style de titre (h1 ? h2 ?) qui reprenne le style du footer de https://www.univ-nantes.fr/, à savoir plus petit, en gras et souligné
  • mettre un logo en haut à gauche et idéalement un bandeau sur toute la partie haute.

Assets utilisés en PJ.

Rendu desktop

image

Rendu mobile

image

Rendu footer

image

Sur la page wcs (https://demarches-univ-nantes.test.entrouvert.org/formulaire-pour-tester-le-rendu-ne-pas-supprimer-ni-modifier/?cancelurl=https%3A//portail-univ-nantes.test.entrouvert.org/page-2-colonnes-1-barre-laterale/) : - [x] pas assez large, on est sur une largeur de 800px, il faut l'augmenter (pas la barre latérale, qui est assez large, mais le contenu) Sur une page combo (https://portail-univ-nantes.test.entrouvert.org/page-2-colonnes-1-barre-laterale/) : - [x] ajouter un peu de largeur sur le contenu (barre latérale ok) - [x] la cellule "lien vers une démarche" manque de padding (cellule dans la colonne de droite, en haut) - [x] dans le footer, il faudrait avoir un style de titre (h1 ? h2 ?) qui reprenne le style du footer de https://www.univ-nantes.fr/, à savoir plus petit, en gras et souligné - [x] mettre un logo en haut à gauche et idéalement un bandeau sur toute la partie haute. Assets utilisés en PJ. # Rendu desktop ![image](/attachments/db933404-8e3d-46aa-9d60-0c5b1ed0923f) # Rendu mobile ![image](/attachments/1738010c-100a-4f25-a7bc-b1698a55ff84) # Rendu footer ![image](/attachments/ebcd7015-acc7-4f62-8b45-77e386c604f0)
image.png
99 KiB
image.png
8.5 KiB
bandeau-intranet.png
136 KiB
Logotype_Nantes-U_blanc_72dpi.png
14 KiB
image.png
257 KiB
bandeau-intranet.png Logotype_Nantes-U_blanc_72dpi.png
csechet requested review from csechet 2022-11-14 11:57:08 +01:00
csechet commented 2022-11-14 12:15:17 +01:00
Owner
Copy Link

Est-ce que tu pourrais fournir le bandeau que tu as utilisé stp ?

Est-ce que tu pourrais fournir le bandeau que tu as utilisé stp ?
aberriot commented 2022-11-14 14:28:25 +01:00
Author
Owner
Copy Link

Est-ce que tu pourrais fournir le bandeau que tu as utilisé stp ?

@csechet il est en pj de mon dernier message, https://gitea.entrouvert.org/attachments/5c03d34e-9ffa-427d-857a-52632f6fd326

> Est-ce que tu pourrais fournir le bandeau que tu as utilisé stp ? @csechet il est en pj de mon dernier message, https://gitea.entrouvert.org/attachments/5c03d34e-9ffa-427d-857a-52632f6fd326
csechet commented 2022-11-14 14:34:27 +01:00
Owner
Copy Link

Est-ce que tu pourrais fournir le bandeau que tu as utilisé stp ?

@csechet il est en pj de mon dernier message, https://gitea.entrouvert.org/attachments/5c03d34e-9ffa-427d-857a-52632f6fd326

Ah ben oui, j'ai pas les yeux en face des trous, merci.

> > Est-ce que tu pourrais fournir le bandeau que tu as utilisé stp ? > > @csechet il est en pj de mon dernier message, https://gitea.entrouvert.org/attachments/5c03d34e-9ffa-427d-857a-52632f6fd326 Ah ben oui, j'ai pas les yeux en face des trous, merci.
csechet requested changes 2022-11-14 14:50:01 +01:00
csechet left a comment
Owner
Copy Link

Pour le reste ça me parait OK, je ferai juste une passe visuelle une fois le slot corrigé, j'y vois pas grand chose sans la banière.

Pour le reste ça me parait OK, je ferai juste une passe visuelle une fois le slot corrigé, j'y vois pas grand chose sans la banière.
static/univ-nantes/config.json Outdated
@ -8,2 +8,3 @@
"COMBO_ASSET_SLOTS.update": {
"header:logo": { "label": "Têtière : logo" }
"header:logo": { "label": "Têtière : logo" },
"header:backgound": { "label" : "Têtière : fond" }
csechet commented 2022-11-14 14:45:54 +01:00
Owner
Copy Link

Typo: c'est header:background, et on peut enlever l'espace insécable, j'imagine.

Typo: c'est header:backg**r**ound, et on peut enlever l'espace insécable, j'imagine.
aberriot commented 2022-11-14 16:05:48 +01:00
Author
Owner
Copy Link

Ah mais c'est pour ça que j'avais des soucis chelous de cache en local. Parfait, merci !

Ah mais c'est pour ça que j'avais des soucis chelous de cache en local. Parfait, merci !
csechet marked this conversation as resolved
csechet force-pushed wip/71216-univ-nantes-retours from 2f14e66382 to a478727a9f 2022-11-14 15:00:11 +01:00 Compare
csechet requested review from csechet 2022-11-14 15:01:17 +01:00
csechet approved these changes 2022-11-14 15:01:25 +01:00
tjund requested review from csechet 2022-11-14 15:11:19 +01:00
tjund requested review from tjund 2022-11-14 15:11:35 +01:00
tjund commented 2022-11-14 15:12:10 +01:00
Owner
Copy Link

Je relis aussi.

Je relis aussi.
👍 1
tjund reviewed 2022-11-14 15:21:38 +01:00
static/univ-nantes/_custom.scss Outdated
@ -52,3 +47,2 @@
#logo {
height: 200px;
@media screen and ($max-mobile-viewport) {
padding-left: unset;
tjund commented 2022-11-14 15:21:38 +01:00
Owner
Copy Link

unset ?
Cette valeur n'apporte que des confusions ici. Elle est égal à initial + inherit.
Padding ne s'hérite pas et sa valeur initiale est 0
Alors autant mettre 0, que tout le monde comprenne.

unset ? Cette valeur n'apporte que des confusions ici. Elle est égal à `initial `+ `inherit`. Padding ne s'hérite pas et sa valeur initiale est 0 Alors autant mettre `0`, que tout le monde comprenne.
aberriot commented 2022-11-16 09:59:31 +01:00
Author
Owner
Copy Link

Effectivement, c'est corrigé

Effectivement, c'est corrigé
aberriot marked this conversation as resolved
tjund reviewed 2022-11-14 15:25:20 +01:00
static/univ-nantes/_custom.scss Outdated
@ -84,3 +77,3 @@
}
a {
color: $black;
color: $white;
tjund commented 2022-11-14 15:25:20 +01:00
Owner
Copy Link

Pourquoi faire compliqué. $black: black; color: $black
white et black c'est bien et ça suffit : color: black
(C'est hors ticket mais j'ai pas pu m'empêcher :)

Pourquoi faire compliqué. `$black: black; color: $black` `white` et `black` c'est bien et ça suffit : `color: black` (C'est hors ticket mais j'ai pas pu m'empêcher :)
aberriot commented 2022-11-16 09:42:12 +01:00
Author
Owner
Copy Link

Comme on a pour convention de mettre les couleurs en variables, ça fait sens pour moi, mais j'entends qu'on simplifie.

Comme on a pour convention de mettre les couleurs en variables, ça fait sens pour moi, mais j'entends qu'on simplifie.
aberriot marked this conversation as resolved
tjund reviewed 2022-11-14 15:32:04 +01:00
tjund left a comment
Owner
Copy Link

Je conseil d'ajouter une background-color au header en plus de l'image (identique au bleu de l'image). Parce que si jamais le client remove l'image du background, on ne voit plus rien.

Je conseil d'ajouter une background-color au header en plus de l'image (identique au bleu de l'image). Parce que si jamais le client remove l'image du background, on ne voit plus rien.
tjund removed review request for csechet 2022-11-14 15:32:50 +01:00
tjund approved these changes 2022-11-14 15:56:03 +01:00
static/univ-nantes/_custom.scss Outdated
@ -102,0 +94,4 @@
box-shadow: 0 0.225em 0.325em rgba(0, 0, 0, 0.1);
position: relative;
top: -76px;
margin-bottom: -76px;
tjund commented 2022-11-14 15:52:18 +01:00
Owner
Copy Link

(Ok, mais dangereux, va casser en cas de zoom "texte seulement").

(Ok, mais dangereux, va casser en cas de zoom "texte seulement").
aberriot commented 2022-11-16 09:56:43 +01:00
Author
Owner
Copy Link

Effectivement, je suis passée sur du em, ça casse pas/très peu sur du zoom texte

Effectivement, je suis passée sur du em, ça casse pas/très peu sur du zoom texte
aberriot marked this conversation as resolved
static/univ-nantes/_custom.scss Outdated
@ -113,0 +131,4 @@
}
body {
@media screen and (max-width: $nav-mobile-limit) {
border-left: 0;
tjund commented 2022-11-14 15:55:52 +01:00
Owner
Copy Link

moi j'utilise l'astuce
$nav-border-color: none
pour supprimer les bordures.
En attendant mieux.

moi j'utilise l'astuce `$nav-border-color: none` pour supprimer les bordures. En attendant mieux.
aberriot commented 2022-11-16 09:56:19 +01:00
Author
Owner
Copy Link

Fait, merci

Fait, merci
aberriot marked this conversation as resolved
tjund requested review from tjund 2022-11-14 15:56:23 +01:00
aberriot force-pushed wip/71216-univ-nantes-retours from a478727a9f to ef750b98ed 2022-11-16 09:56:05 +01:00 Compare
aberriot force-pushed wip/71216-univ-nantes-retours from ef750b98ed to 7f24181b18 2022-11-16 09:57:50 +01:00 Compare
aberriot force-pushed wip/71216-univ-nantes-retours from 7f24181b18 to e4457dcd08 2022-11-16 09:59:24 +01:00 Compare
aberriot commented 2022-11-16 10:02:38 +01:00
Author
Owner
Copy Link

Mergé avec les correctifs liés à vos retours :)

Mergé avec les correctifs liés à vos retours :)
aberriot merged commit d12341c2f1 into main 2022-11-16 10:02:49 +01:00
aberriot deleted branch wip/71216-univ-nantes-retours 2022-11-16 10:02:49 +01:00
tjund approved these changes 2022-11-16 10:12:00 +01:00
static/univ-nantes/_custom.scss
@ -84,3 +78,3 @@
}
a {
color: $black;
color: $white;
tjund commented 2022-11-16 10:10:16 +01:00
Owner
Copy Link

Tu as oublié un $white ici, et comme la varible n'existe plus.

Tu as oublié un $white ici, et comme la varible n'existe plus.
Sign in to join this conversation.
Reviewers
No reviewers
csechet
tjund
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
3 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: entrouvert/publik-base-theme#10
No description provided.
Delete Branch "wip/71216-univ-nantes-retours"

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?