scss: allow nav item text transformation (#72361) #75

Merged
smihai merged 2 commits from wip/72361-allow-nav-items-transformation into main 2022-12-14 15:38:52 +01:00
Owner
No description provided.
smihai added 1 commit 2022-12-13 16:49:39 +01:00
gitea-wip/publik-base-theme/pipeline/pr-main This commit looks good Details
cb1108c9f1
scss: allow nav items text transformation (#72361)
aberriot reviewed 2022-12-14 08:22:32 +01:00
aberriot left a comment
Owner

Ah trop bien cette nouvelle variable. Je me demande s'il ne peut pas y avoir un effet de bord par contre sur des thèmes qui auraient fait un truc genre

div.gru-nav li a { text-transform: x }, ça risque pas de sauter vu la spécificité du sélecteur qui passe à none maintenant ?

Ah trop bien cette nouvelle variable. Je me demande s'il ne peut pas y avoir un effet de bord par contre sur des thèmes qui auraient fait un truc genre `div.gru-nav li a { text-transform: x }`, ça risque pas de sauter vu la spécificité du sélecteur qui passe à none maintenant ?
Author
Owner

En effet, ça peut impacter les thèmes où une transformation est appliquée aux items. Tout dépend comment cette transformation a été faite. Je fais un tour des thèmes.

En effet, ça peut impacter les thèmes où une transformation est appliquée aux items. Tout dépend comment cette transformation a été faite. Je fais un tour des thèmes.
Owner
Ça saute pour ces endroits là au moins (j'ai cherché/testé rapidement) : - https://gitea.entrouvert.org/entrouvert/publik-base-theme/src/branch/main/static/calvados-intranet-cd14/_custom.scss#L169 - https://gitea.entrouvert.org/entrouvert/publik-base-theme/src/branch/main/static/calvados-cd14/_custom.scss#L69 - https://gitea.entrouvert.org/entrouvert/publik-base-theme/src/branch/main/static/clapiers/_custom.scss#L45 - https://gitea.entrouvert.org/entrouvert/publik-base-theme/src/branch/main/static/cournonterral/_custom.scss#L30
Owner

peut-être qu'en déplaçant la variable dans le sélecteur de moins spécifique possible, genre :

.site-nav {
  text-transform: $nav-item-transform;
}

Ça peut résoudre le souci ou limiter fortement les effets de bord sur les thèmes existants ?

peut-être qu'en déplaçant la variable dans le sélecteur de moins spécifique possible, genre : ```css .site-nav { text-transform: $nav-item-transform; } ``` Ça peut résoudre le souci ou limiter fortement les effets de bord sur les thèmes existants ?
Author
Owner

peut-être qu'en déplaçant la variable dans le sélecteur de moins spécifique possible, genre :

.site-nav {
  text-transform: $nav-item-transform;
}

Ça peut résoudre le souci ou limiter fortement les effets de bord sur les thèmes existants ?

Pour moi il faut faire une passe sur tous les thèmes.

> peut-être qu'en déplaçant la variable dans le sélecteur de moins spécifique possible, genre : > ```css > .site-nav { > text-transform: $nav-item-transform; > } > ``` > > Ça peut résoudre le souci ou limiter fortement les effets de bord sur les thèmes existants ? Pour moi il faut faire une passe sur tous les thèmes.
smihai added 1 commit 2022-12-14 12:13:47 +01:00
gitea-wip/publik-base-theme/pipeline/pr-main This commit looks good Details
bf334472e3
scss: apply new nav item's transformation variable to themes
Author
Owner

Je viens d'ajouter un commit avec passe sur tous les thèmes.

Je viens d'ajouter un commit avec passe sur tous les thèmes.
aberriot approved these changes 2022-12-14 15:11:58 +01:00
aberriot left a comment
Owner

Sacré boulot, relu, okay pour moi !

Sacré boulot, relu, okay pour moi !
smihai merged commit a366ad7d35 into main 2022-12-14 15:38:52 +01:00
smihai deleted branch wip/72361-allow-nav-items-transformation 2022-12-14 15:38:52 +01:00
Sign in to join this conversation.
No reviewers
No Label
No Milestone
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#75
No description provided.