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

universite-toulon: add theme (#74558) #170

Merged
csechet merged 1 commits from wip/74558-theme-toulon into main 2023-02-15 18:01:49 +01:00
Conversation 3 Commits 1 Files Changed 18 +880
csechet commented 2023-02-15 15:59:08 +01:00
Owner
Copy Link
No description provided.
wcs-mobile.png
308 KiB
wcs-desktop.png
374 KiB
combo-desktop.png
595 KiB
combo-mobile.png
272 KiB
wcs-mobile.png wcs-desktop.png combo-desktop.png combo-mobile.png
aberriot requested review from aberriot 2023-02-15 16:00:10 +01:00
csechet commented 2023-02-15 16:08:15 +01:00
Author
Owner
Copy Link
No description provided.
export_page_index_20230215.json
19 KiB
aberriot approved these changes 2023-02-15 17:00:59 +01:00
aberriot left a comment
Owner
Copy Link

Ça tourne très bien en local et c'est fidèle à https://sitetest.univ-tln.fr/. Trucs que j'ai remarqué :

Le bloc « Une idée de démarche à nous proposer ? » manque de spacing avec la sidebar sur les pages courtes:

image

Je vois un soulignement sous les titres des cellules en sidebar, qui ne semble pas matcher le site de démo (sauf pour celui de la dernière cellule). J'aurai tendance à tous les enlever.

image

La liste de lien du footer contient des séparateurs blancs qui ne sont pas présents sur le site de démo :

image

Je marque comme validé parce que c'est non bloquant à mon sens.

Ça tourne très bien en local et c'est fidèle à https://sitetest.univ-tln.fr/. Trucs que j'ai remarqué : Le bloc « Une idée de démarche à nous proposer ? » manque de spacing avec la sidebar sur les pages courtes: ![image](/attachments/e5a8be58-3a54-44e7-ab70-f679afc545c5) Je vois un soulignement sous les titres des cellules en sidebar, qui ne semble pas matcher le site de démo (sauf pour celui de la dernière cellule). J'aurai tendance à tous les enlever. ![image](/attachments/2fac26fc-afbf-4449-8c37-c1e43aee0d9f) La liste de lien du footer contient des séparateurs blancs qui ne sont pas présents sur le site de démo : ![image](/attachments/dab3d73a-1cd3-4a3a-a6b4-eb07da88aa56) Je marque comme validé parce que c'est non bloquant à mon sens.
image.png
60 KiB
image.png
87 KiB
image.png
12 KiB
static/universite-toulon/_vars.scss Outdated
@ -0,0 +17,4 @@
$border-radius-1: 10px;
$border-radius-2: 3px;
$box-shadow-1: 0 0 5px 2px $gray-5;
aberriot commented 2023-02-15 16:30:40 +01:00
Owner
Copy Link

Je vois que ce box shadow s'applique aux inputs en focus (par exemple sur la page de connexion). Ça n'est pas sur leur site de démo, on peut peut-être l'enlever des inputs ?

Je vois que ce box shadow s'applique aux inputs en focus (par exemple sur la page de connexion). Ça n'est pas sur leur site de démo, on peut peut-être l'enlever des inputs ?
static/universite-toulon/_vars.scss Outdated
@ -0,0 +142,4 @@
// footer
$footer-background: $primary-color;
aberriot commented 2023-02-15 16:47:25 +01:00
Owner
Copy Link

Visiblement, c'est un bleu un peu plus clair, #0698D6 qui est utilisé sur le site de démo.

Visiblement, c'est un bleu un peu plus clair, #0698D6 qui est utilisé sur le site de démo.
static/universite-toulon/_wcs.scss Outdated
@ -0,0 +4,4 @@
&:hover {
box-shadow: none;
}
}
aberriot commented 2023-02-15 16:58:15 +01:00
Owner
Copy Link

Je ne sais pas si on veut en tenir compte, mais les boutons suivant / précédent de leur site de démo utilisent un bleu plus clair, cf https://sitetest.univ-tln.fr/formulaire/

Je ne sais pas si on veut en tenir compte, mais les boutons suivant / précédent de leur site de démo utilisent un bleu plus clair, cf https://sitetest.univ-tln.fr/formulaire/
templates/variants/universite-toulon/theme.html Outdated
@ -0,0 +1,12 @@
{% extends 'theme.html' %}
{# move nav #}
aberriot commented 2023-02-15 17:00:40 +01:00
Owner
Copy Link

Super stratégie pour bouger la nav sans faire des trucs horribles en CSS, je ne connaissais pas, merci !

Super stratégie pour bouger la nav sans faire des trucs horribles en CSS, je ne connaissais pas, merci !
csechet force-pushed wip/74558-theme-toulon from 6a297ef2a1 to ceb0b9e325 2023-02-15 17:51:35 +01:00 Compare
csechet commented 2023-02-15 17:51:46 +01:00
Author
Owner
Copy Link

Pour le contexte, la maquette a été faite par une personne qui n'est pas UX ou graphiste, j'ai pris quelques libertés, notamment :

  • Utiliser un bleu légèrement plus foncé pour avoir un contraste de 4.5 avec le blanc, ça n'était pas suffisant avec les couleurs fournies pour respecter les WCAG.
  • J'ai unifié différentes choses qui étaient disparates ou pas cohérentes sur la maquettes, dont : les soulignements de titres de cellules, la couleur des boutons (certains boutons en hover bleu clair et d'autre bleu clair inactif est confusant, je l'ai juste laissé sur WCS où il n'y a que des boutons clairs), la présentation des link-list entre le contenu et le footer (d'où les séparations), les carets sont aussi les mêmes entre les links list du contenu et celles du footer.
  • Ajouter un retour visuel pour le focus, absent de la maquette, j'ai choisi une shadow box semblable à celle du thème Armentières, comme elle s'est appuyée dessus pour faire sa maquette.

Donc je vais laisser en l'état tout ces points, et je règle le problème avec la cellule suggestion.

Pour le contexte, la maquette a été faite par une personne qui n'est pas UX ou graphiste, j'ai pris quelques libertés, notamment : * Utiliser un bleu légèrement plus foncé pour avoir un contraste de 4.5 avec le blanc, ça n'était pas suffisant avec les couleurs fournies pour respecter les WCAG. * J'ai unifié différentes choses qui étaient disparates ou pas cohérentes sur la maquettes, dont : les soulignements de titres de cellules, la couleur des boutons (certains boutons en hover bleu clair et d'autre bleu clair inactif est confusant, je l'ai juste laissé sur WCS où il n'y a que des boutons clairs), la présentation des link-list entre le contenu et le footer (d'où les séparations), les carets sont aussi les mêmes entre les links list du contenu et celles du footer. * Ajouter un retour visuel pour le focus, absent de la maquette, j'ai choisi une shadow box semblable à celle du thème Armentières, comme elle s'est appuyée dessus pour faire sa maquette. Donc je vais laisser en l'état tout ces points, et je règle le problème avec la cellule suggestion.
👍 1
csechet merged commit 4f34a69f3b into main 2023-02-15 18:01:49 +01:00
aberriot commented 2023-02-16 09:05:57 +01:00
Owner
Copy Link

Ça fait sens, merci pour les clarifications !

Ça fait sens, merci pour les clarifications !
Sign in to join this conversation.
Reviewers
No reviewers
aberriot
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
2 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#170
No description provided.
Delete Branch "wip/74558-theme-toulon"

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?