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

scss: fix radio & checkbox lists with long labels padding (#79326) #353

Merged
csechet merged 1 commits from wip/79326-mauvais-rendu-en-bouton into main 2023-07-20 14:41:08 +02:00
Conversation 2 Commits 1 Files Changed 1 +2 -1
csechet commented 2023-07-18 16:24:47 +02:00
Owner
Copy Link
No description provided.
csechet added 1 commit 2023-07-18 16:24:48 +02:00
gitea/publik-base-theme/pipeline/head This commit looks good Details
9a4dc47d80 scss: fix radio & checkbox lists with long labels padding (#79326)
aberriot approved these changes 2023-07-18 16:34:54 +02:00
aberriot left a comment
Owner
Copy Link

dans le ticket il est dit « Lorsque les libellés des items d'une liste sont longs, le rendu en boutons radio n'est pas terrible: l'espace vertical entre la première et deuxième lignes est plus important que celui les autres (du au margin-bottom du input), les lignes des libellés pourraient être indentés. », je ne vois rien concernant l'indentation, c'est considéré comme hors scope ?

dans le ticket il est dit « Lorsque les libellés des items d'une liste sont longs, le rendu en boutons radio n'est pas terrible: l'espace vertical entre la première et deuxième lignes est plus important que celui les autres (du au margin-bottom du input), les lignes des libellés pourraient être indentés. », je ne vois rien concernant l'indentation, c'est considéré comme hors scope ?
csechet commented 2023-07-19 14:56:46 +02:00
Author
Owner
Copy Link

dans le ticket il est dit « Lorsque les libellés des items d'une liste sont longs, le rendu en boutons radio n'est pas terrible: l'espace vertical entre la première et deuxième lignes est plus important que celui les autres (du au margin-bottom du input), les lignes des libellés pourraient être indentés. », je ne vois rien concernant l'indentation, c'est considéré comme hors scope ?

Effectivement : il est indiqué "pourrait" et c'est compliqué de le faire sans changer le markup (et donc prendre des risques de casser des thèmes) : je ne l'ai pas fait du coup.

> dans le ticket il est dit « Lorsque les libellés des items d'une liste sont longs, le rendu en boutons radio n'est pas terrible: l'espace vertical entre la première et deuxième lignes est plus important que celui les autres (du au margin-bottom du input), les lignes des libellés pourraient être indentés. », je ne vois rien concernant l'indentation, c'est considéré comme hors scope ? > Effectivement : il est indiqué "pourrait" et c'est compliqué de le faire sans changer le markup (et donc prendre des risques de casser des thèmes) : je ne l'ai pas fait du coup.
smihai commented 2023-07-20 09:30:23 +02:00
Owner
Copy Link

Effectivement : il est indiqué "pourrait" et c'est compliqué de le faire sans changer le markup (et donc prendre des risques de casser des thèmes) : je ne l'ai pas fait du coup.

T'apportes déjà une correction, donc ça me va.

> > Effectivement : il est indiqué "pourrait" et c'est compliqué de le faire sans changer le markup (et donc prendre des risques de casser des thèmes) : je ne l'ai pas fait du coup. T'apportes déjà une correction, donc ça me va.
csechet merged commit 1c1cc28e65 into main 2023-07-20 14:41:08 +02:00
csechet deleted branch wip/79326-mauvais-rendu-en-bouton 2023-07-20 14:41:08 +02:00
Sign in to join this conversation.
Reviewers
No reviewers
aberriot
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#353
No description provided.
Delete Branch "wip/79326-mauvais-rendu-en-bouton"

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?