fix_icons #20

Merged
thubrecht merged 2 commits from fix_icons into main 2025-08-01 00:36:36 +02:00
Owner
No description provided.
Owner

les icones sont devenues un peu petites peut-être non ?
image

les icones sont devenues un peu petites peut-être non ? ![image](/attachments/9e4a4649-5911-419f-b499-f432eab9c892)
7.7 KiB
thubrecht closed this pull request 2025-07-27 17:55:59 +02:00
mdebray reopened this pull request 2025-07-27 18:47:49 +02:00
Author
Owner

Le fix proposé est moins complet que celui-ci. En particulier si on set la taille de police sans utiliser les helpers bulma ça ne marche pas

Le fix proposé est moins complet que celui-ci. En particulier si on set la taille de police sans utiliser les helpers bulma ça ne marche pas
Owner

@mdebray wrote in #20 (comment):

Le fix proposé est moins complet que celui-ci. En particulier si on set la taille de police sans utiliser les helpers bulma ça ne marche pas

Ton css passe l'intégralité des icônes en minuscule sans discrimination, le fix sur main correspond au comportement précédent dans les cas normaux

@mdebray wrote in https://git.dgnum.eu/DGNum/dgnum.eu/pulls/20#issuecomment-16852: > Le fix proposé est moins complet que celui-ci. En particulier si on set la taille de police sans utiliser les helpers bulma ça ne marche pas Ton css passe l'intégralité des icônes en minuscule sans discrimination, le fix sur main correspond au comportement précédent dans les cas normaux
fix: make theme toggle icon bigger
All checks were successful
Deploy dgnum.eu / deploy (push) Successful in 29s
edbb4d4b0f
This commit also aligns correctly the icon in the navbar
Author
Owner

J'ai pas bien compris ton commentaire. J'utilise des em et pas des rem. Perso sur firefox j'ai:

Avant (88939090):
image

Mnt (5117de5d):
image

(on remarque un léger misalignement car bulma essaie de faire des choses intelligentes avec la classe icon dans un navbar-item, c'était le point de départ de cette PR)

Avec ma version du fix (ece6d86):
image

De plus en changeant la taille de police les icônes changent effectivement de taille.

Enfin, je pense que utiliser de em pour la taille est le bon fix. En particulier,sur main, si je fais:

<p class="is-size-1">
<span class="icon-text"><span class="icon"><i data-ti="sun"></i></span><span>Foobar</span> llorem ipsum
</p>

J'obtiens une icône trop petite car elle ne suit pas la font-size:
image

J'ai poussé une version du fix qui réaligne horizontalement l'icône de navbar tout en le mettant plus gros.

J'ai pas bien compris ton commentaire. J'utilise des em et pas des rem. Perso sur firefox j'ai: Avant (88939090): ![image](/attachments/54929831-9661-4fa6-9140-bc34f7509adb) Mnt (5117de5d): ![image](/attachments/2a933b00-c156-454e-9a41-7d6df5b7a1f1) (on remarque un léger misalignement car bulma essaie de faire des choses intelligentes avec la classe `icon` dans un `navbar-item`, c'était le point de départ de cette PR) Avec ma version du fix (ece6d86): ![image](/attachments/9c5ae469-775d-45fc-9144-03c399f1d1fa) De plus en changeant la taille de police les icônes changent effectivement de taille. Enfin, je pense que utiliser de em pour la taille est le bon fix. En particulier,sur main, si je fais: ``` <p class="is-size-1"> <span class="icon-text"><span class="icon"><i data-ti="sun"></i></span><span>Foobar</span> llorem ipsum </p> ``` J'obtiens une icône trop petite car elle ne suit pas la font-size: ![image](/attachments/e9d5b6be-d831-4f3b-8981-abf367bbeb22) J'ai poussé une version du fix qui réaligne horizontalement l'icône de navbar tout en le mettant plus gros.
Owner

Il est vrai, j'ai raconté des bêtises

Il est vrai, j'ai raconté des bêtises
thubrecht approved these changes 2025-08-01 00:36:21 +02:00
thubrecht deleted branch fix_icons 2025-08-01 00:36:36 +02:00
Sign in to join this conversation.
No reviewers
No milestone
No project
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: DGNum/dgnum.eu#20
No description provided.