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

css: remove specific padding on .pk-button (#74837) #180

Merged
tjund merged 1 commits from wip/74837-same-padding-button-pk-button into main 2023-03-03 09:32:17 +01:00
Conversation 5 Commits 1 Files Changed 2 +1 -6
tjund commented 2023-02-24 12:15:00 +01:00
Owner
Copy Link
No description provided.
fpeters commented 2023-02-24 12:54:49 +01:00
Owner
Copy Link

C'est prévu à la base pour les cellules texte, ça fait déjà aujourd'hui des lignes plus hautes que la normale, là ça va accentuer encore les choses.

image

On a une petite idée de cet usage vs je ne suis pas trop sûr de quoi parce que quand je suis les tickets j'arrive à une proposition de réduire la hauteur de boutons (qui sonne comme l'inverse de l'impact annoncé,

Impact : tous les .pk-button vont gagner par défaut 4px en hauteur.

C'est prévu à la base pour les cellules texte, ça fait déjà aujourd'hui des lignes plus hautes que la normale, là ça va accentuer encore les choses. ![image](/attachments/366ba801-703c-4ce5-87b6-41a839beeee9) On a une petite idée de cet usage vs je ne suis pas trop sûr de quoi parce que quand je suis les tickets j'arrive à une proposition de réduire la hauteur de boutons (qui sonne comme l'inverse de l'impact annoncé, > Impact : tous les .pk-button vont gagner par défaut 4px en hauteur.
image.png
12 KiB
fpeters commented 2023-02-24 12:58:17 +01:00
Owner
Copy Link

Et le fonctionnement actuel donne chez moi Clapotis un rendu où le pk-button posé dans une cellule texte (cf capture plus haut) a la même hauteur que le bouton d'une démarche :

image

Ici si le rendu dans combo augmente, et pas l'autre, ça ne va plus aller.

Et le fonctionnement actuel donne chez moi Clapotis un rendu où le pk-button posé dans une cellule texte (cf capture plus haut) a la même hauteur que le bouton d'une démarche : ![image](/attachments/d027fd66-eca4-4ab6-bd86-b9330360c77b) Ici si le rendu dans combo augmente, et pas l'autre, ça ne va plus aller.
image.png
4.0 KiB
tjund commented 2023-02-24 15:17:54 +01:00
Author
Owner
Copy Link

C'est prévu à la base pour les cellules texte

Oui mais comme ce n'est plus le cas : .pk-button est utilisé pour mettre en forme les cellules liens, ajouter des boutons dans les cellules json, cellules fiches, et pourrait être utilisé ailleurs.

ça fait déjà aujourd'hui des lignes plus hautes que la normale

Oui, c'est donc une mauvaise idée que d'insérer un bouton au sein d'un paragraphe, quelle que soit sa taille.

quand je suis les tickets j'arrive à une proposition de réduire la hauteur de boutons (qui sonne comme l'inverse de l'impact annoncé,

Dans le cas de Toulouse, la charte proposait de larges paddings sur les widgets et %buttons, le besoin est de revenir à la normale, mais partout le même "normal": que .pk-button et %buttons soient identiques. Et je me suis demandé si ce ne serait pas logique finalement que ce soit le comportement par défaut.

Et le fonctionnement actuel donne chez moi Clapotis un rendu où le pk-button posé dans une cellule texte (cf capture plus haut) a la même hauteur que le bouton d'une démarche

Oui, l'idée est d'uniformiser.

Ici si le rendu dans combo augmente, et pas l'autre, ça ne va plus aller.

Désolé, je ne comprends pas, que veux-tu dire ?

> C'est prévu à la base pour les cellules texte Oui mais comme ce n'est plus le cas : .pk-button est utilisé pour mettre en forme les cellules liens, ajouter des boutons dans les cellules json, cellules fiches, et pourrait être utilisé ailleurs. > ça fait déjà aujourd'hui des lignes plus hautes que la normale Oui, c'est donc une mauvaise idée que d'insérer un bouton au sein d'un paragraphe, quelle que soit sa taille. > quand je suis les tickets j'arrive à une proposition de réduire la hauteur de boutons (qui sonne comme l'inverse de l'impact annoncé, Dans le cas de Toulouse, la charte proposait de larges paddings sur les widgets et %buttons, le besoin est de revenir à la normale, mais partout le même "normal": que .pk-button et %buttons soient identiques. Et je me suis demandé si ce ne serait pas logique finalement que ce soit le comportement par défaut. > Et le fonctionnement actuel donne chez moi Clapotis un rendu où le pk-button posé dans une cellule texte (cf capture plus haut) a la même hauteur que le bouton d'une démarche Oui, l'idée est d'uniformiser. > Ici si le rendu dans combo augmente, et pas l'autre, ça ne va plus aller. Désolé, je ne comprends pas, que veux-tu dire ?
fpeters commented 2023-02-24 16:25:06 +01:00
Owner
Copy Link

Désolé, je ne comprends pas, que veux-tu dire ?

Chez moi (clapotis) .pk-button et bouton w.c.s. ont la même hauteur; comme ce ticket annonce que .pk-button va gagner 8px, je me dis qu'ils seront 8px plus haut que les boutons w.c.s.

> Désolé, je ne comprends pas, que veux-tu dire ? Chez moi (clapotis) .pk-button et bouton w.c.s. ont la même hauteur; comme ce ticket annonce que .pk-button va gagner 8px, je me dis qu'ils seront 8px plus haut que les boutons w.c.s.
tjund force-pushed wip/74837-same-padding-button-pk-button from 1c045dfd76 to c3ffcfe963 2023-03-02 09:18:12 +01:00 Compare
tjund commented 2023-03-02 09:25:42 +01:00
Author
Owner
Copy Link

Énigme résolue.
La différence de padding est compensé par un line-height différent.
l'element button a un line-height 'normal' alors que a.pk-button hérite du line-height défini par $font-line-height
Les 2 boutons ont donc une hauteur similaire avec un font-size de 16, et c'est un hasard.
Donc en uniformisant le line-height, les 2 boutons resteront identiques, même si $font-size ou $font-line-height sont personnalisés.

Patch modifié

Énigme résolue. La différence de padding est compensé par un line-height différent. l'element button a un line-height 'normal' alors que a.pk-button hérite du line-height défini par $font-line-height Les 2 boutons ont donc une hauteur similaire avec un font-size de 16, et c'est un hasard. Donc en uniformisant le line-height, les 2 boutons resteront identiques, même si $font-size ou $font-line-height sont personnalisés. Patch modifié
tjund requested review from fpeters 2023-03-03 09:24:36 +01:00
fpeters approved these changes 2023-03-03 09:25:35 +01:00
tjund merged commit 8584c64351 into main 2023-03-03 09:32:17 +01:00
tjund deleted branch wip/74837-same-padding-button-pk-button 2023-03-03 09:32:18 +01:00
Sign in to join this conversation.
Reviewers
No reviewers
fpeters
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#180
No description provided.
Delete Branch "wip/74837-same-padding-button-pk-button"

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?