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

ehess: add initial theme (#76923) #272

Merged
pmarillonnet merged 1 commits from wip/76923-ehess-initial-theme into main 2023-05-30 16:34:00 +02:00
Conversation 5 Commits 1 Files Changed 5 +213
pmarillonnet commented 2023-05-17 12:37:35 +02:00
Owner
Copy Link

Gitea me dit que j’ai supprimé la branche, la précédente PR a été fermée, je ne peux pas la rouvrir. Le mystère plane et comme je n’ai pas le temps de creuser, j’ouvre une autre PR.

Gitea me dit que j’ai supprimé la branche, la précédente PR a été fermée, je ne peux pas la rouvrir. Le mystère plane et comme je n’ai pas le temps de creuser, j’ouvre une autre PR.
pmarillonnet force-pushed wip/76923-ehess-initial-theme from b9b28a76a7 to 8a1c3455e5 2023-05-17 15:45:34 +02:00 Compare
pmarillonnet force-pushed wip/76923-ehess-initial-theme from 8a1c3455e5 to 9e2da2eab5 2023-05-17 16:20:17 +02:00 Compare
pmarillonnet force-pushed wip/76923-ehess-initial-theme from 9e2da2eab5 to 0ee14b0b95 2023-05-17 16:30:28 +02:00 Compare
pmarillonnet force-pushed wip/76923-ehess-initial-theme from 0ee14b0b95 to ba8f771e1e 2023-05-22 09:34:24 +02:00 Compare
pmarillonnet force-pushed wip/76923-ehess-initial-theme from ba8f771e1e to 5c7297b13b 2023-05-22 15:59:48 +02:00 Compare
pmarillonnet force-pushed wip/76923-ehess-initial-theme from 5c7297b13b to f690dabff9 2023-05-24 10:58:16 +02:00 Compare
pmarillonnet force-pushed wip/76923-ehess-initial-theme from f690dabff9 to 819c018ece 2023-05-24 12:02:04 +02:00 Compare
pmarillonnet force-pushed wip/76923-ehess-initial-theme from 819c018ece to 3c8b4ab3c7 2023-05-24 16:32:39 +02:00 Compare
pmarillonnet force-pushed wip/76923-ehess-initial-theme from 3c8b4ab3c7 to 6ac68dc850 2023-05-25 09:33:49 +02:00 Compare
pmarillonnet force-pushed wip/76923-ehess-initial-theme from 6ac68dc850 to d929ad37e7 2023-05-25 10:11:02 +02:00 Compare
pmarillonnet force-pushed wip/76923-ehess-initial-theme from d929ad37e7 to ebb55dd068 2023-05-25 10:33:00 +02:00 Compare
pmarillonnet changed title from WIP: ehess: add initial theme (#76923) to ehess: add initial theme (#76923) 2023-05-25 10:33:10 +02:00
pmarillonnet commented 2023-05-25 10:34:46 +02:00
Author
Owner
Copy Link

Une approche un peu minimaliste/brutaliste, fortuit concours de circonstance entre l’apparence du site institutionnel et mes compétences en CSS :)

Avec quelques captures pour une idée du rendu.

Une approche un peu minimaliste/brutaliste, fortuit concours de circonstance entre l’apparence du site institutionnel et mes compétences en CSS :) Avec quelques captures pour une idée du rendu.
accounts-wide.png
130 KiB
form-wide.png
519 KiB
portal-narrow.png
227 KiB
portal-wide.png
249 KiB
accounts-narrow.png
98 KiB
style-narrow.png
349 KiB
form-narrow.png
381 KiB
style-wide.png
357 KiB
accounts-wide.png form-wide.png portal-narrow.png portal-wide.png accounts-narrow.png style-narrow.png form-narrow.png style-wide.png
pmarillonnet commented 2023-05-25 10:36:24 +02:00
Author
Owner
Copy Link

Pour rappel on se réfère à https://dev.entrouvert.org/attachments/75506 mais aussi au site institutionnel qui déroge un peu à ça (la police principale notamment qui n’est pas Work Sans comme indiqué dans la charte, mais Public Sans).

Pour rappel on se réfère à https://dev.entrouvert.org/attachments/75506 mais aussi au site institutionnel qui déroge un peu à ça (la police principale notamment qui n’est pas Work Sans comme indiqué dans la charte, mais Public Sans).
csechet requested review from csechet 2023-05-30 15:39:52 +02:00
csechet approved these changes 2023-05-30 16:22:07 +02:00
csechet left a comment
Owner
Copy Link

J'ai pushé des petites suggestions et activé le menu mobile : ça peut être déroutant pour le client d'avoir à configurer les pages de la navigation mobile via le B.O séparément de la nav desktop, mais s'ils sont OK avec ça à toi de voir si tu garde ou pas. Pour les autres modifs, c'est une histoire de goûts : c'est tout aussi facultatif :)

J'ai pushé des petites suggestions et activé le menu mobile : ça peut être déroutant pour le client d'avoir à configurer les pages de la navigation mobile via le B.O séparément de la nav desktop, mais s'ils sont OK avec ça à toi de voir si tu garde ou pas. Pour les autres modifs, c'est une histoire de goûts : c'est tout aussi facultatif :)
csechet commented 2023-05-30 16:24:03 +02:00
Owner
Copy Link

Ah et il manque du contraste entre le vert école et le fond blanc, mais comme c'est directement comme ça dans leur charte graphique et que c'est pas moi qui vais aller discuter avec eux de ce genre de problèmes, je vais pas être tatillon là-dessus.

Ah et il manque du contraste entre le vert école et le fond blanc, mais comme c'est directement comme ça dans leur charte graphique et que c'est pas moi qui vais aller discuter avec eux de ce genre de problèmes, je vais pas être tatillon là-dessus.
pmarillonnet commented 2023-05-30 16:30:34 +02:00
Author
Owner
Copy Link

J'ai pushé des petites suggestions et activé le menu mobile : ça peut être déroutant pour le client d'avoir à configurer les pages de la navigation mobile via le B.O séparément de la nav desktop, mais s'ils sont OK avec ça à toi de voir si tu garde ou pas. Pour les autres modifs, c'est une histoire de goûts : c'est tout aussi facultatif :)

Merci beaucoup, je squash tout ça et je fusionne dans main.

> J'ai pushé des petites suggestions et activé le menu mobile : ça peut être déroutant pour le client d'avoir à configurer les pages de la navigation mobile via le B.O séparément de la nav desktop, mais s'ils sont OK avec ça à toi de voir si tu garde ou pas. Pour les autres modifs, c'est une histoire de goûts : c'est tout aussi facultatif :) Merci beaucoup, je squash tout ça et je fusionne dans main.
pmarillonnet commented 2023-05-30 16:32:32 +02:00
Author
Owner
Copy Link

Ah et il manque du contraste entre le vert école et le fond blanc, mais comme c'est directement comme ça dans leur charte graphique et que c'est pas moi qui vais aller discuter avec eux de ce genre de problèmes, je vais pas être tatillon là-dessus.

Je suis d’avis qu’on passe déjà ce thème raccord avec leur charte. En parallèle je vais informer Clément, CPF du projet, de ce souci de contraste, on verra si on le leur rapporte.

> Ah et il manque du contraste entre le vert école et le fond blanc, mais comme c'est directement comme ça dans leur charte graphique et que c'est pas moi qui vais aller discuter avec eux de ce genre de problèmes, je vais pas être tatillon là-dessus. Je suis d’avis qu’on passe déjà ce thème raccord avec leur charte. En parallèle je vais informer Clément, CPF du projet, de ce souci de contraste, on verra si on le leur rapporte.
pmarillonnet force-pushed wip/76923-ehess-initial-theme from d831384597 to 3a75810d6b 2023-05-30 16:33:22 +02:00 Compare
pmarillonnet merged commit 3a75810d6b into main 2023-05-30 16:34:00 +02:00
Sign in to join this conversation.
Reviewers
No reviewers
csechet
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#272
No description provided.
Delete Branch "wip/76923-ehess-initial-theme"

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?