Project

General

Profile

Bug #53817

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

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

Status:
Solution déployée
Priority:
Normal
Assignee:
Category:
-
Target version:
-
Start date:
06 May 2021
Due date:
% Done:

0%

Estimated time:
Patch proposed:
Yes
Planning:
No

Description

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


Files

Associated revisions

Revision d66778cf (diff)
Added by Thomas Jund about 1 month ago

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

History

#1

Updated by Thomas Jund about 1 month ago

#2

Updated by Nicolas Roche about 1 month ago

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

Updated by Thomas Jund about 1 month ago

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

Updated by Nicolas Roche about 1 month ago

(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

Updated by Thomas Jund about 1 month ago

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

#8

Updated by Serghei Mihai about 1 month ago

  • Status changed from Solution proposée to Solution validée
#9

Updated by Thomas Jund about 1 month ago

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

Updated by Frédéric Péters about 1 month ago

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

Also available in: Atom PDF