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

est-ensemble: add initial theme (#74222) #163

Merged
smihai merged 2 commits from wip/74222-add-est-ensemble-theme into main 2023-02-10 10:02:50 +01:00
Conversation 0 Commits 2 Files Changed 4 +136
smihai commented 2023-02-07 11:54:11 +01:00
Owner
Copy Link
No description provided.
form.png
555 KiB
desktop.png
1.8 MiB
mobile.png
821 KiB
form.png desktop.png mobile.png
smihai added 1 commit 2023-02-07 11:54:12 +01:00
gitea-wip/publik-base-theme/pipeline/pr-main This commit looks good Details
2129753867 est-ensemble: add initial theme (#74222)
csechet requested review from csechet 2023-02-07 13:00:26 +01:00
csechet added 1 commit 2023-02-08 17:54:12 +01:00
gitea-wip/publik-base-theme/pipeline/pr-main This commit looks good Details
34768b217c avoid tag usage in selectors
csechet reviewed 2023-02-08 17:54:47 +01:00
csechet left a comment
Owner
Copy Link

Deux questions, et j'ai pushé une proposition pour éviter d'utiliser les tags et utiliser les nouveaux sélecteurs BEM disponibles pour la nav.

Deux questions, et j'ai pushé une proposition pour éviter d'utiliser les tags et utiliser les nouveaux sélecteurs BEM disponibles pour la nav.
static/est-ensemble/_custom.scss
@ -0,0 +24,4 @@
.gru-nav-button {
position: absolute;
&.toggled {
position: fixed;
csechet commented 2023-02-08 17:48:26 +01:00
Owner
Copy Link

Je n'ai pas bien compris à quoi servait de mettre le bouton en fixed quand il était activé.

Je n'ai pas bien compris à quoi servait de mettre le bouton en fixed quand il était activé.
smihai commented 2023-02-09 09:45:47 +01:00
Author
Owner
Copy Link

Avec position: absolute le menu burger n'est pas scrollable.
Et avec position: fixed je souhaite qu'il le devienne lorsqu'il le menu mobile est ouvert.

Avec `position: absolute` le menu burger n'est pas scrollable. Et avec `position: fixed` je souhaite qu'il le devienne lorsqu'il le menu mobile est ouvert.
csechet marked this conversation as resolved
static/est-ensemble/_custom.scss
@ -0,0 +28,4 @@
}
}
> ul.menu {
@media (min-width: $nav-mobile-limit + 1) {
csechet commented 2023-02-08 17:49:12 +01:00
Owner
Copy Link

Je n'ai pas compris non plus pourquoi +1.

Je n'ai pas compris non plus pourquoi +1.
smihai commented 2023-02-09 09:47:11 +01:00
Author
Owner
Copy Link

Pour appliquer les styles lorsqu'on dépasse la media query. C'est ce qu'on fait à peu près partout, il me semble.

Pour appliquer les styles lorsqu'on dépasse la media query. C'est ce qu'on fait à peu près partout, il me semble.
csechet marked this conversation as resolved
csechet approved these changes 2023-02-10 10:02:04 +01:00
smihai merged commit a1cdc96afc into main 2023-02-10 10:02:50 +01:00
smihai deleted branch wip/74222-add-est-ensemble-theme 2023-02-10 10:02:51 +01: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#163
No description provided.
Delete Branch "wip/74222-add-est-ensemble-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?