Projet

Général

Profil

Bug #53817

CSS "width 100%" déclaré 7x sur .agenda-table

Ajouté par Thomas Jund (congés, retour le 29/04) il y a presque 3 ans. Mis à jour il y a presque 3 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Thomas Jund (congés, retour le 29/04)
Catégorie:
-
Version cible:
-
Début:
06 mai 2021
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

parce que le sélecteur et la propriété est posé dans une boucle.

Fichiers

0001-css-don-t-loop-width-prop-on-.agenda-table-53817.patch (793 octets) 0001-css-don-t-loop-width-prop-on-.agenda-table-53817.patch Thomas Jund (congés, retour le 29/04), 06 mai 2021 15:15
0001-css-remove-dead-code-bad-selector-53817.patch (3,86 ko) 0001-css-remove-dead-code-bad-selector-53817.patch Thomas Jund (congés, retour le 29/04), 06 mai 2021 16:02
0002-css-don-t-loop-width-prop-on-.agenda-table-53817.patch (828 octets) 0002-css-don-t-loop-width-prop-on-.agenda-table-53817.patch Thomas Jund (congés, retour le 29/04), 10 mai 2021 16:36
0001-css-don-t-loop-width-prop-on-.agenda-table-53817.patch (824 octets) 0001-css-don-t-loop-width-prop-on-.agenda-table-53817.patch Thomas Jund (congés, retour le 29/04), 10 mai 2021 18:24

Révisions associées

Révision d66778cf (diff)
Ajouté par Thomas Jund (congés, retour le 29/04) il y a presque 3 ans

css: don't loop width prop on .agenda-table (#53817)

Historique

#1

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

#2

Mis à jour par Nicolas Roche il y a presque 3 ans

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

Ack.

chrono/manager/static/css/style.css

<<<
.agenda-table {
  width: 100%; }
  .agenda-table .desks-1 thead th {
    width: 99%; }

.agenda-table {
  width: 100%; }
  .agenda-table .desks-2 thead th {
    width: 49%; }

.agenda-table {
  width: 100%; }
  .agenda-table .desks-3 thead th {
    width: 32.3333333333%; }

.agenda-table {
  width: 100%; }
  .agenda-table .desks-4 thead th {
    width: 24%; }

.agenda-table {
  width: 100%; }
  .agenda-table .desks-5 thead th {
    width: 19%; }

.agenda-table {
  width: 100%; }
  .agenda-table .desks-6 thead th {
    width: 15.6666666667%; }

.agenda-table {
  width: 100%; }
  .agenda-table .desks-7 thead th {
    width: 13.2857142857%; }
===
.agenda-table {
  width: 100%; }
  .agenda-table .desks-1 thead th {
    width: 99%; }
  .agenda-table .desks-2 thead th {
    width: 49%; }
  .agenda-table .desks-3 thead th {
    width: 32.3333333333%; }
  .agenda-table .desks-4 thead th {
    width: 24%; }
  .agenda-table .desks-5 thead th {
    width: 19%; }
  .agenda-table .desks-6 thead th {
    width: 15.6666666667%; }
  .agenda-table .desks-7 thead th {
    width: 13.2857142857%; }
>>>

#3

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

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

En fait c'est même plus que cela, la boucle ne sert à rien :

@for $i from 1 through 7 {
    .agenda-table {
        width: 100%;
        .desks-#{$i} {
            thead th { width: (100%/$i)-1%; }
        }
    }
}

retourne `.agenda-table .desks-* thead th {}` qui cible dans le vide.

Je propose donc de supprimer.

#5

Mis à jour par Nicolas Roche il y a presque 3 ans

(patch avec trop de trucs dedans)

J'imagine que ce qui avait été voulu sur #21326 c'était :

diff --git a/chrono/manager/static/css/style.scss b/chrono/manager/static/css/style.scss
index ba8c5b8..64d312b 100644
--- a/chrono/manager/static/css/style.scss
+++ b/chrono/manager/static/css/style.scss
@@ -134,8 +134,9 @@ table.agenda-table {

+.agenda-table {
+       width: 100%;
+}
+
 @for $i from 1 through 7 {
-       .agenda-table {
-               width: 100%;
-               .desks-#{$i} {
-                       thead th { width: (100%/$i)-1%; }
-               }
+       .agenda-table.desks-#{$i} {
+               thead th { width: (100%/$i)-1%; }
        }

Mais j'avoue que je ne vois pas la différence à l'affichage.

edit: ça semble avoir sauté ici https://dev.entrouvert.org/issues/21326#note-64

#6

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

T'as raison, je patch le mauvais sélecteur aussi, fixer un width sur les desks est finalement utile.

#8

Mis à jour par Serghei Mihai il y a presque 3 ans

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

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

  • Statut changé de Solution validée à Résolu (à déployer)
commit d66778cfb81121aa12bd6466a51813d03433786f
Author: Thomas JUND <tjund@entrouvert.com>
Date:   Mon May 10 16:29:03 2021 +0200

    css: don't loop width prop on .agenda-table (#53817)
#10

Mis à jour par Frédéric Péters il y a presque 3 ans

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

Formats disponibles : Atom PDF