Project

General

Profile

Bug #39073

$border-radius en doublon. 3 ou 0 ?

Added by Thomas Jund about 1 month ago. Updated 16 days ago.

Status:
Solution déployée
Priority:
Normal
Assignee:
Target version:
-
Start date:
17 Jan 2020
Due date:
% Done:

0%

Patch proposed:
Yes
Planning:
No

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.

0001-scss-remove-duplicate-border-radius-declaration-3907.patch View (1.22 KB) Thomas Jund, 17 Jan 2020 03:55 PM

0001-scss-remove-duplicate-border-radius-declaration-3907.patch View (9.37 KB) Thomas Jund, 24 Jan 2020 03:10 PM


Related issues

Related to Intégrations graphiques Publik - Development #39199: Intégration graphique pour Saint-Laurent-du-Var Solution déployée 23 Jan 2020

Associated revisions

Revision 3f12c65e (diff)
Added by Thomas Jund 24 days ago

scss: remove duplicate $border-radius declaration (#39073)

History

#1 Updated by Thomas Jund about 1 month ago

Laisser la valeur par défaut à zéro me parait plus logique.

#2 Updated by Thomas Jund 28 days ago

#3 Updated by Nicolas Roche 28 days ago

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

#4 Updated by Frédéric Péters 27 days ago

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...

#5 Updated by Thomas Jund 27 days ago

É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.

#6 Updated by Nicolas Roche 27 days ago

  • Status changed from Solution proposée to 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.

#9 Updated by Thomas Jund 24 days ago

  • Status changed from Solution validée to 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)

#10 Updated by Frédéric Péters 16 days ago

  • Status changed from Résolu (à déployer) to Solution déployée

Also available in: Atom PDF