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

bulk: clear form's sections from floating elements (#82720) #419

Merged
smihai merged 1 commits from wip/82642-bulk-clear-forms-sections-from-floating-elements into main 2023-10-23 15:05:02 +02:00
Conversation 0 Commits 1 Files Changed 1 +4
smihai commented 2023-10-23 13:57:18 +02:00
Owner
Copy Link
No description provided.
smihai added 1 commit 2023-10-23 13:57:19 +02:00
gitea/publik-base-theme/pipeline/head This commit looks good Details
ebde7746bf bulk: clear form's sections from floating elements (#82720)
tjund requested changes 2023-10-23 14:50:39 +02:00
static/includes/wcs/_bulk.scss Outdated
@ -765,6 +765,10 @@ p.use-file-from-fargo {
margin-top: 0;
}
div.section {
tjund commented 2023-10-23 14:50:31 +02:00
Owner
Copy Link

Je n'aime pas trop ce selecteur, à la lecture du code, on ne comprend pas de quoi il s'agit

Je n'aime pas trop ce selecteur, à la lecture du code, on ne comprend pas de quoi il s'agit
tjund commented 2023-10-23 14:57:30 +02:00
Owner
Copy Link

Mais je n'ai rien de mieux à proposer, alors oublions

Mais je n'ai rien de mieux à proposer, alors oublions
smihai commented 2023-10-23 14:59:49 +02:00
Author
Owner
Copy Link

Je n'aime pas trop ce selecteur, à la lecture du code, on ne comprend pas de quoi il s'agit

Je suis d'accord avec toi, mais je ne me vois pas modifier les noms des classes (pas ici en tout cas).

> Je n'aime pas trop ce selecteur, à la lecture du code, on ne comprend pas de quoi il s'agit Je suis d'accord avec toi, mais je ne me vois pas modifier les noms des classes (pas ici en tout cas).
smihai marked this conversation as resolved
static/includes/wcs/_bulk.scss Outdated
@ -766,2 +766,4 @@
}
div.section {
clear: both;
tjund commented 2023-10-23 14:50:00 +02:00
Owner
Copy Link

Il est préférable d'utliser un clearfix comme

@extend .clearfix;

Il est préférable d'utliser un clearfix comme @extend .clearfix;
smihai commented 2023-10-23 15:00:14 +02:00
Author
Owner
Copy Link

Il est préférable d'utliser un clearfix comme

@extend .clearfix;

Bien vu, fait.

> Il est préférable d'utliser un clearfix comme > > @extend .clearfix; Bien vu, fait.
smihai marked this conversation as resolved
smihai force-pushed wip/82642-bulk-clear-forms-sections-from-floating-elements from ebde7746bf to c5bc82fa87 2023-10-23 14:58:50 +02:00 Compare
smihai requested review from tjund 2023-10-23 15:01:20 +02:00
tjund approved these changes 2023-10-23 15:02:18 +02:00
smihai merged commit c9a0e978a0 into main 2023-10-23 15:05:02 +02:00
smihai deleted branch wip/82642-bulk-clear-forms-sections-from-floating-elements 2023-10-23 15:05:02 +02:00
Sign in to join this conversation.
Reviewers
No reviewers
tjund
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#419
No description provided.
Delete Branch "wip/82642-bulk-clear-forms-sections-from-floating-elements"

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?