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

vars: add nav font related variables (#77709) #275

Merged
pmarillonnet merged 1 commits from wip/77709-nav-font-related-vars into main 2023-05-17 16:30:01 +02:00
Conversation 1 Commits 1 Files Changed 1 +8 -1
pmarillonnet commented 2023-05-17 15:47:04 +02:00
Owner
Copy Link

Testé en local, ça m’a l’air de fonctionner.

Testé en local, ça m’a l’air de fonctionner.
smihai reviewed 2023-05-17 15:51:36 +02:00
static/includes/_nav.scss Outdated
@ -4,1 +4,4 @@
$nav-border-radius: $border-radius !default;
$nav-font-family: $font-family !default;
$nav-font-style: normal !default;
$nav-font-size: 1.1 * $font-size !default;
smihai commented 2023-05-17 15:51:23 +02:00
Owner
Copy Link

Tu augmentes ici la taille de la police pour toutes les integrations, non?

Tu augmentes ici la taille de la police pour toutes les integrations, non?
pmarillonnet commented 2023-05-17 15:54:48 +02:00
Author
Owner
Copy Link

Oui, c’est vrai :D
J’ai posé cela par mimétisme de la façon dont est définie $title-font-size, mais c’est une mauvaise idée, je vais laisser la valeur de taille de police globale par défaut.

Oui, c’est vrai :D J’ai posé cela par mimétisme de la façon dont est définie `$title-font-size`, mais c’est une mauvaise idée, je vais laisser la valeur de taille de police globale par défaut.
pmarillonnet commented 2023-05-17 15:56:08 +02:00
Author
Owner
Copy Link

C’est corrigé, merci pour ta vigilance.

C’est corrigé, merci pour ta vigilance.
pmarillonnet force-pushed wip/77709-nav-font-related-vars from 202cb8694e to a48f99a8f3 2023-05-17 15:55:52 +02:00 Compare
smihai approved these changes 2023-05-17 15:56:54 +02:00
smihai left a comment
Owner
Copy Link

Nickel

Nickel
pmarillonnet force-pushed wip/77709-nav-font-related-vars from a48f99a8f3 to 34deed3991 2023-05-17 16:13:25 +02:00 Compare
pmarillonnet force-pushed wip/77709-nav-font-related-vars from 34deed3991 to d8fc037768 2023-05-17 16:16:33 +02:00 Compare
pmarillonnet force-pushed wip/77709-nav-font-related-vars from d8fc037768 to 5221e31d62 2023-05-17 16:18:52 +02:00 Compare
pmarillonnet commented 2023-05-17 16:23:26 +02:00
Author
Owner
Copy Link

Nickel

Et entre temps j’ai ajouté la variable correspondant au font-weight css. Testé en local et ça m’a l’air de fonctionner aussi.

> Nickel Et entre temps j’ai ajouté la variable correspondant au `font-weight` css. Testé en local et ça m’a l’air de fonctionner aussi.
pmarillonnet requested review from smihai 2023-05-17 16:23:30 +02:00
pmarillonnet added 1 commit 2023-05-17 16:25:57 +02:00
gitea/publik-base-theme/pipeline/head This commit looks good Details
24d98a9293 vars: add nav font related variables (#77709)
smihai approved these changes 2023-05-17 16:29:32 +02:00
pmarillonnet merged commit 24d98a9293 into main 2023-05-17 16:30:01 +02:00
Sign in to join this conversation.
Reviewers
No reviewers
smihai
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#275
No description provided.
Delete Branch "wip/77709-nav-font-related-vars"

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?