rouen-intranet: add theme (#74913) #184

Merged
aberriot merged 1 commits from wip/74913-rouen into main 2023-03-01 09:52:06 +01:00
Owner

Validé en visio pour la partie fonctionnelle avec Brice.

Le gros du CSS custom est consacré au header / menu pour s'approcher de l'existant (https://d-clic.rouen.fr/fr/index.html, se connecter avec les identifiants indiqués sur #66965).

image

image

image

Validé en visio pour la partie fonctionnelle avec Brice. Le gros du CSS custom est consacré au header / menu pour s'approcher de l'existant (https://d-clic.rouen.fr/fr/index.html, se connecter avec les identifiants indiqués sur #66965). ![image](/attachments/09749e99-6b1a-4136-8eef-bea68329f537) ![image](/attachments/a3454942-2164-43ef-a344-3b5a1cd171ff) ![image](/attachments/bbe1b9d3-5290-48c1-8b84-432f41b2ab6a)
tjund reviewed 2023-02-28 18:08:57 +01:00
@ -0,0 +89,4 @@
#toplinks {
z-index: 999;
}
main *:not(.cell) .cell {
Owner

Tu as des cas où tu as une cellule dans cellule ?

Tu as des cas où tu as une cellule dans cellule ?
Author
Owner

ça m'est arrivé sur une home page avec des grilles imbriquées il me semble, mais je ne saurai pas reproduire

ça m'est arrivé sur une home page avec des grilles imbriquées il me semble, mais je ne saurai pas reproduire
tjund reviewed 2023-03-01 09:07:46 +01:00
@ -0,0 +16,4 @@
$font-size: 18px;
$font-line-height: 1.5em;
$base-font: 16;
Owner

Pour que $fz-1: 30em / $base-font; soit un equivalent em de 30px, il faut que $base-font soit équivalement à $font-size, donc 18.

Pour que `$fz-1: 30em / $base-font;` soit un equivalent em de 30px, il faut que $base-font soit équivalement à $font-size, donc 18.
Author
Owner

merci, je viens de pousser un fix

merci, je viens de pousser un fix
aberriot marked this conversation as resolved
tjund reviewed 2023-03-01 09:27:35 +01:00
@ -0,0 +28,4 @@
$font-family: "Open Sans", sans-serif;
$width: 1160px;
$font-color: #000;
$font-size: 0.938em;
Owner

tu redéclares $font-size ici, du coup je ne sais pas…

tu redéclares $font-size ici, du coup je ne sais pas…
Author
Owner

Erreur de copier coller de ma part, j'ai viré celui ci et repassé le premier sur 16 (et adapté le base font)

Erreur de copier coller de ma part, j'ai viré celui ci et repassé le premier sur 16 (et adapté le base font)
aberriot marked this conversation as resolved
Owner

Sinon, "pas d'enjeu" (dixit Brice) donc je valide.

Sinon, "pas d'enjeu" (dixit Brice) donc je valide.
tjund approved these changes 2023-03-01 09:42:55 +01:00
aberriot force-pushed wip/74913-rouen from 8e723e88a8 to 8bfe1c935c 2023-03-01 09:50:36 +01:00 Compare
aberriot merged commit b5dcbbd32c into main 2023-03-01 09:52:06 +01:00
aberriot deleted branch wip/74913-rouen 2023-03-01 09:52:06 +01:00
Sign in to join this conversation.
No reviewers
No Label
No Milestone
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#184
No description provided.