Projet

Général

Profil

Bug #39158

$cell-border-radius non appliqué

Ajouté par Thomas Jund (congés, retour le 29/04) il y a plus de 4 ans. Mis à jour il y a environ 4 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Thomas Jund (congés, retour le 29/04)
Version cible:
-
Début:
21 janvier 2020
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

$cell-border-radius n'est pas appliqué sur .cell

voir image bug1

mais appliquer la bonne variable avec

div.a2-continue,
div#services > ul > li,
div.a2-block,
.gru-content div.cell,
div.block {
    text-align: left;
    background: $cell-background;
    color: $cell-color;
    box-sizing: border-box;
-    border-radius: $border-radius;
+    border-radius: $cell-border-radius;
    border: $cell-border;
}

ne suffit pas. Des bugs de dépassements sont visibles (voir image bug2).

Des patchs pour éviter ces dépassements sont présents l.206 à l.217 mais n prennent pas en compte tous les problèmes

Je propose de les remplacer par un simple `overflow: hidden` directement sur `.cell` (voir image patch)

Fichiers

patch.png (879 ko) patch.png Thomas Jund (congés, retour le 29/04), 21 janvier 2020 17:23
Bugs2.png (906 ko) Bugs2.png Thomas Jund (congés, retour le 29/04), 21 janvier 2020 17:23
bug1.png (1,05 Mo) bug1.png Thomas Jund (congés, retour le 29/04), 21 janvier 2020 17:23
0001-scss-cells-improve-cells-with-cell-border-radius-391.patch (1,32 ko) 0001-scss-cells-improve-cells-with-cell-border-radius-391.patch Thomas Jund (congés, retour le 29/04), 21 janvier 2020 17:42

Demandes liées

Lié à Intégrations graphiques Publik - Development #39199: Intégration graphique pour Saint-Laurent-du-VarFermé23 janvier 2020

Actions
Lié à Intégrations graphiques Publik - Bug #40261: Conserver overflow-hidden sur les .cell ou restaurer les hacks ?Fermé28 février 2020

Actions

Révisions associées

Révision e7286688 (diff)
Ajouté par Thomas Jund (congés, retour le 29/04) il y a environ 4 ans

scss: cells: improve cells with $cell-border-radius (#39158)

Révision 64466d9a (diff)
Ajouté par Frédéric Péters il y a environ 4 ans

quimper: force square borders (#39862)

(broken by #39158)

Révision 7b9d26f1 (diff)
Ajouté par Frédéric Péters il y a environ 4 ans

quimper: force square borders (#39862)

(broken by #39158)

Historique

#1

Mis à jour par Thomas Jund (congés, retour le 29/04) il y a plus de 4 ans

  • Description mis à jour (diff)
#2

Mis à jour par Thomas Jund (congés, retour le 29/04) il y a plus de 4 ans

#3

Mis à jour par Thomas Jund (congés, retour le 29/04) il y a plus de 4 ans

#4

Mis à jour par Frédéric Péters il y a environ 4 ans

  • Statut changé de Solution proposée à Solution validée
#5

Mis à jour par Thomas Jund (congés, retour le 29/04) il y a environ 4 ans

  • Statut changé de Solution validée à Résolu (à déployer)

Vu que par défaut $border-radius et donc $cell-border-radius est `null`, j'ai simplifié la condition

- @if ($cell-border-radius != 0) {
+ @if $cell-border-radius {

commit e7286688e26280096a117c28c107dba3739e8e4e (HEAD -> master, origin/master, origin/HEAD)
Author: Thomas JUND <tjund@entrouvert.com>
Date:   Tue Jan 21 17:34:16 2020 +0100

    scss: cells: improve cells with $cell-border-radius (#39158)
#6

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

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é

Formats disponibles : Atom PDF