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

scss: introduce $edge-gutter & $edge-gutter-mobile (#85848) #495

Merged
tjund merged 1 commits from wip/85848-edge-gutter into main 2024-03-20 15:03:22 +01:00
Conversation 4 Commits 1 Files Changed 4 +76 -15
tjund commented 2024-01-18 17:08:32 +01:00
Owner
Copy Link
No description provided.
tjund added 1 commit 2024-01-18 17:08:32 +01:00
gitea/publik-base-theme/pipeline/head This commit looks good Details
56cc656ec7 scss: introduce $edge-gutter & $edge-gutter-mobile (#85848)
fpeters commented 2024-01-18 18:02:13 +01:00
Owner
Copy Link

C'est pour moi confus d'avoir une variable qui est à la fois un booléen et une mesure: "Espace (« gouttière ») entre le contenu et le bord du navigateur.". Quand on met "false" j'imaginerais que ça voudrait dire 0, soit "pas d'espace", et que "pas d'espace" ça voudrait dire que le contenu colle au bord de l'écran, c'est-à-dire que toute la largeur soit prise, ce qui n'est je pense pas le cas parce qu'on contrôle par ailleurs la largeur du contenu.

C'est pour moi confus d'avoir une variable qui est à la fois un booléen et une mesure: "Espace (« gouttière ») entre le contenu et le bord du navigateur.". Quand on met "false" j'imaginerais que ça voudrait dire 0, soit "pas d'espace", et que "pas d'espace" ça voudrait dire que le contenu colle au bord de l'écran, c'est-à-dire que toute la largeur soit prise, ce qui n'est je pense pas le cas parce qu'on contrôle par ailleurs la largeur du contenu.
tjund commented 2024-01-29 09:38:07 +01:00
Author
Owner
Copy Link

ok, False est un mauvais choix. Je vais plutôt nommer cela "legacy"

ok, False est un mauvais choix. Je vais plutôt nommer cela "legacy"
tjund force-pushed wip/85848-edge-gutter from 56cc656ec7 to 432d4e2cf8 2024-02-06 15:03:33 +01:00 Compare
tjund commented 2024-02-06 16:11:39 +01:00
Author
Owner
Copy Link

MàJ avec check des pages wcs et authentic.

MàJ avec check des pages wcs et authentic.
smihai approved these changes 2024-03-19 17:11:52 +01:00
help/fr/misc-scss.page Outdated
@ -149,0 +149,4 @@
<tr>
<td><p><code>$edge-gutter</code></p></td>
<td><p>Espace (« gouttière ») entre le contenu et le bord du navigateur.</p></td>
<td><p><var>false</var></p></td>
smihai commented 2024-03-19 17:10:39 +01:00
Owner
Copy Link

remplacer ici l'exemple "false" par "legacy" et go.

remplacer ici l'exemple "false" par "legacy" et go.
tjund force-pushed wip/85848-edge-gutter from 432d4e2cf8 to db3812d5b0 2024-03-20 14:57:37 +01:00 Compare
tjund force-pushed wip/85848-edge-gutter from db3812d5b0 to f8da9e5d1f 2024-03-20 15:00:44 +01:00 Compare
tjund commented 2024-03-20 15:01:45 +01:00
Author
Owner
Copy Link

Merci, changement de la doc ok.

Merci, changement de la doc ok.
tjund merged commit e139dff369 into main 2024-03-20 15:03:22 +01:00
tjund deleted branch wip/85848-edge-gutter 2024-03-20 15:03:22 +01: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
3 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#495
No description provided.
Delete Branch "wip/85848-edge-gutter"

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?