Bug #39883
condition @if $cell-border-radius tout le temps (?) vérifiée
0%
Description
À traiter #39862 je me rends compte que
@if $cell-border-radius {
border-radius: $cell-border-radius;
overflow: hidden;
}
était ajouté même avec un cell-border-radius à 0. Ça ne me semble pas l'effet souhaité.
Il faudrait mettre $cell-border-radius à null pour ne pas entrer dans la condition mais ça n'est pas une valeur CSS valide, donc non.
Fichiers
Demandes liées
Révisions associées
scss: fix check for positive $cell-border-radius (#39883)
Historique
Mis à jour par Thomas Jund (congés, retour le 29/04) il y a environ 4 ans
`null` est la valeur sass la plus intéressante pour $border-radius par défaut (je pense).
$cell-border-radius hérite de la valeur de $border-radius.
Donc dans les cas où on a une valeur à $border-radius et on veut spécifiquement un reset de $cell-border-radius à 0 il faut écrire en effet `$cell-border-radius: null`, qui n'est en effet pas une valeur CSS.
retourner false sur `null`, `0` (et `0px`), `initial` et `unset` ?
Mis à jour par Thomas Jund (congés, retour le 29/04) il y a environ 4 ans
- Fichier 0001-scss-check-if-cell-border-radius-is-a-counting-numbe.patch 0001-scss-check-if-cell-border-radius-is-a-counting-numbe.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Assigné à mis à Thomas Jund (congés, retour le 29/04)
- Patch proposed changé de Non à Oui
Mis à jour par Thomas Jund (congés, retour le 29/04) il y a environ 4 ans
Side comment:
En travaillant sur cette solution, je me suis encore demandé si `overflow-hidden` était finalement la meilleure solution, c'est la solution "officielle" et la plus propre pour empêcher les enfants possédant un backgroud et des borders de dépasser. Mais je me suis rendu compte qu'elle engendrait encore des régressions sur les thèmes existants, mais également qu'elle annulait l'option `$cell-title-cover-border`.
Mis à jour par Frédéric Péters il y a environ 4 ans
Indenter avec une tabulation.
qu'elle engendrait encore des régressions sur les thèmes existants
Soit on continue à les corriger, soit on arrête et on retourne comme c'était avant, mais il n'y a pas de ticket concernant ces régressions ?
Mis à jour par Thomas Jund (congés, retour le 29/04) il y a environ 4 ans
Soit on continue à les corriger, soit on arrête et on retourne comme c'était avant, mais il n'y a pas de ticket concernant ces régressions ?
Soit je trouve un moyen d'améliorer le core pour éviter de patcher les thèmes existants un à un (mais c'est vrai que ce matin je n'y crois plus trop.
Retourner comme c'était avant, c'est restaurer tous les hacks qui compliquaient le code et qui n'apportaient pas de solutions à plusieurs cas de figures.
J'ouvre un ticket pour lister mes investigations.
Mis à jour par Thomas Jund (congés, retour le 29/04) il y a environ 4 ans
- Lié à Bug #40261: Conserver overflow-hidden sur les .cell ou restaurer les hacks ? ajouté
Mis à jour par Frédéric Péters il y a environ 4 ans
- Statut changé de Solution proposée à Résolu (à déployer)
Parce que je viens encore de trouver un problème à cause de l'overflow: hidden (accessibilité clavier de toodego en partie pétée); parce que ne pas l'appliquer quand il n'y a pas de bordure arrondie est la chose correcte à faire, je viens d'appliquer ce patch, après avoir changé "counting number" en "positive number" et corrigé le commentaire associé.
commit 87482fc3f193d3080b5e3db31385653b470c212f
Author: Thomas JUND <tjund@entrouvert.com>
Date: Thu Feb 27 15:44:47 2020 +0100
scss: fix check for positive $cell-border-radius (#39883)
On parle d'un problème présent depuis pile 1 mois en recette, passé en prod et pas corrigé dans le cycle qui suivi, ça relativiserait l'importance du problème mais j'ai quand même ça en hotfix.
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: fix check for positive $cell-border-radius (#39883)