Bug #39073
$border-radius en doublon. 3 ou 0 ?
0%
Description
la variable $border-radius est déclarée 2x
- _general.scss l.1: `$border-radius: 3px !default;`
- _nav.scss l.36: `$border-radius: 0 !default;`
la valeur `3px` dans _general.scss prend l'ascendant car située avant.
Pourtant dans la doc, cette valeur est indiquée comme devant être à 0 par défaut.
Fichiers
Demandes liées
Révisions associées
Historique
Mis à jour par Thomas Jund il y a environ 4 ans
- Fichier 0001-scss-remove-duplicate-border-radius-declaration-3907.patch 0001-scss-remove-duplicate-border-radius-declaration-3907.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Laisser la valeur par défaut à zéro me parait plus logique.
Mis à jour par Thomas Jund il y a environ 4 ans
- Lié à Development #39199: Intégration graphique pour Saint-Laurent-du-Var ajouté
Mis à jour par Nicolas Roche il y a environ 4 ans
Laisser la valeur par défaut à zéro me parait plus logique.
oui, mais cela va légèrement changer le visuel des thèmes qui utilisent cette valeur par défaut.
(comme ci-dessous, ceux qui ne la redéfinissent pas dans leur fichier _vars.scss
)
$ find . -name '_vars.scss' -exec grep -Hc '$border-radius:' {} \; | grep ':0$' ./static/metz-metropole-2019/_vars.scss:0 ./static/sens/_vars.scss:0 ./static/montreuil/_vars.scss:0 ./static/saone-et-loire-cd71-2019/_vars.scss:0 ./static/gers-cd32/_vars.scss:0 ./static/toulouse-metropole/_vars.scss:0 ./static/bouches-du-rhone/_vars.scss:0 ./static/nancy-en-direct/_vars.scss:0 ./static/venissieux/_vars.scss:0 ./static/lomme/_vars.scss:0 ./static/alpes-maritimes-2018/_vars.scss:0 ./static/tours-metropole/_vars.scss:0 ./static/clermont-metropole/_vars.scss:0 ./static/toulouse/_vars.scss:0 ./static/fontenay-sous-bois-2018/_vars.scss:0 ./static/saint-denis/_vars.scss:0 ./static/lille/_vars.scss:0 ./static/quimper/_vars.scss:0 ./static/hellemmes/_vars.scss:0 ./static/vendargues/_vars.scss:0
Mis à jour par Frédéric Péters il y a environ 4 ans
Comme posé par Nicolas, il faudrait ajuster les intégrations graphiques qui s'attendaient à avoir 3px pour que ça y soit explicitement indiqué. Je n'ai pas regardé si la liste de Nicolas était correcte, et parfois ça peut sans doute ne pas être fait, par exemple je regardais Quimper et comme il n'y a pas de couleur de fond aux titres, le côté "bordure" est de toute façon invisible...
Mis à jour par Thomas Jund il y a environ 4 ans
- Fichier 0001-scss-remove-duplicate-border-radius-declaration-3907.patch 0001-scss-remove-duplicate-border-radius-declaration-3907.patch ajouté
Étant donné que $border-radius est utilisé dans la nav, cellules, titres, boutons, steps, j'ai appliqué la valeur de 3px dans tous les thèmes qui ne l'avaient pas personnalisé.
Si 0 est la valeur ok par défaut, je me suis permis de la passer à `null` pour omettre les déclarations dans ce cas.
Mis à jour par Nicolas Roche il y a environ 4 ans
- Statut changé de Solution proposée à Solution validée
Bien vu, j'avais manqué Thonon (où la ligne était présente mais commentée).
$ find . -name '_vars.scss' -exec grep -Hc '^$border-radius:' {} \; | grep ':0$' (est bien vide à présent)
Si 0 est la valeur ok par défaut, je me suis permis de la passer à `null` pour omettre les déclarations dans ce cas.
yep, en effet je ne la voit plus dans firebug.
Mis à jour par Thomas Jund il y a environ 4 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 3f12c65efcaa678e6e8721461387ca5f497caf89 (HEAD -> master, origin/master, origin/HEAD) Author: Thomas JUND <tjund@entrouvert.com> Date: Fri Jan 17 15:55:30 2020 +0100 scss: remove duplicate $border-radius declaration (#39073)
Mis à jour par Frédéric Péters il y a environ 4 ans
- Statut changé de Résolu (à déployer) à Solution déployée
scss: remove duplicate $border-radius declaration (#39073)