Project

General

Profile

Development #24255

la hague : classe pour avoir des bordures aux cellules

Added by Frédéric Péters about 1 year ago. Updated 5 months ago.

Status:
Rejeté
Priority:
Normal
Assignee:
Josué Kouka
Start date:
04 Jun 2018
Due date:
% Done:

0%

Patch proposed:
Yes
Planning:
No

Description

De #23241 , « déclarer un style no-border ou un truc dans le genre qui permette de décider quand on les veut ou non [les bordures] »

0001-cells-add-no-border-class-24255.patch View (1.15 KB) Josué Kouka, 04 Jun 2018 02:19 PM

no_boder.png View (27.7 KB) Josué Kouka, 04 Jun 2018 02:20 PM

0002-lahague-revert-to-default-border-style-24255.patch View (643 Bytes) Josué Kouka, 04 Jun 2018 03:28 PM

0001-cells-add-no-border-class-24255.patch View (1.16 KB) Josué Kouka, 04 Jun 2018 03:28 PM

0001-cells-add-no-border-and-with-border-classes-24255.patch View (1.57 KB) Josué Kouka, 05 Jun 2018 10:47 AM

borders.png View (20.6 KB) Josué Kouka, 05 Jun 2018 10:49 AM

0001-cells-add-with-border-class-24255.patch View (1.36 KB) Josué Kouka, 05 Jun 2018 06:00 PM

0001-lahague-add-class-to-add-cell-border-24255.patch View (704 Bytes) Josué Kouka, 06 Jun 2018 02:41 PM

lahague_cell_border.png View (277 KB) Josué Kouka, 06 Jun 2018 02:41 PM

0001-lahague-add-class-to-add-cell-border-24255.patch View (628 Bytes) Josué Kouka, 12 Jun 2018 09:37 AM

0001-lahague-add-class-to-add-cell-border-24255.patch View (704 Bytes) Josué Kouka, 18 Jul 2018 11:12 AM

25176
25240
25288

History

#1 Updated by Josué Kouka about 1 year ago

  • Status changed from Nouveau to En cours

#2 Updated by Josué Kouka about 1 year ago

25176

#3 Updated by Frédéric Péters about 1 year ago

Et maintenant réfléchir à la situation. Se rappeler que La Hague n'affiche par défaut pas de bordures..

#5 Updated by Frédéric Péters about 1 year ago

Nope. On n'a pas envie d'imposer aux admins la mention d'une classe sur toutes les cellules.

#6 Updated by Josué Kouka about 1 year ago

Frédéric Péters a écrit :

Nope. On n'a pas envie d'imposer aux admins la mention d'une classe sur toutes les cellules.

Je ne vois pas trop comment je peux rendre ça paramétrable pour toutes les cellules d'une instance.

#7 Updated by Serghei Mihai about 1 year ago

Il me semble qu'il ne s'agit pas de le faire pour toutes les cellules.
Avoir une classe du genre "with-border" qui permet d'afficher la bordure pour une cellule.

#8 Updated by Josué Kouka about 1 year ago

Serghei Mihai a écrit :

Il me semble qu'il ne s'agit pas de le faire pour toutes les cellules.
Avoir une classe du genre "with-border" qui permet d'afficher la bordure pour une cellule.

Je pense que c'est ce que le patch fait. Sauf si je rate un truc, mais pour l'instant je ne vois pas quoi

#9 Updated by Frédéric Péters about 1 year ago

Je pense que c'est ce que le patch fait. Sauf si je rate un truc, mais pour l'instant je ne vois pas quoi

Tu rates sans doute que le revert n'est pas désiré, le rendu par défaut doit être sans bordures. Et sur certaines cellules, il y a souhait de pouvoir poser une classe pour avoir des bordures.

#10 Updated by Josué Kouka about 1 year ago

25240

Bon, j'ai finalement fait 2 classes. L'une (with-border) qui ajoute des bordures quand par défaut on en a pas et l'autre (no-border) quand on en a pas par defaut.
Une capture d'écran de mon test quand par défaut on a pas de bordures.

#11 Updated by Serghei Mihai about 1 year ago

Mais y a pas besoin de classe "no-border" puisque par défaut les cellules n'ont pas de bordure: #23260

#12 Updated by Josué Kouka about 1 year ago

Serghei Mihai a écrit :

Mais y a pas besoin de classe "no-border" puisque par défaut les cellules n'ont pas de bordure: #23260

C'est aussi pour prévoir le cas inverse.

#13 Updated by Serghei Mihai about 1 year ago

Quel cas inverse ? Des cellules sans bordures auxquelles il faut retirer la bordure?

#14 Updated by Josué Kouka about 1 year ago

Serghei Mihai a écrit :

Quel cas inverse ? Des cellules sans bordures auxquelles il faut retirer la bordure?

Non, des cellules avec bordures auxquelles on voudrait retirer des bordures.

#15 Updated by Serghei Mihai about 1 year ago

Mais il n'y a plus de bordure sur les cellules depuis le commit dans ##23260.

#16 Updated by Josué Kouka about 1 year ago

Serghei Mihai a écrit :

Mais il n'y a plus de bordure sur les cellules depuis le commit dans ##23260.

Ok. (Meme si l'on n'ajoute pas cette classe juste pour La Hague).
Bon j'ajoute une classe with-border qui rajoute une bordure a une cellule.

#17 Updated by Frédéric Péters about 1 year ago

Josué, abandonne l'idée de faire un truc utile partout, tape juste une classe côté la hague et c'est tout.

#18 Updated by Josué Kouka about 1 year ago

25288

Frédéric Péters a écrit :

Josué, abandonne l'idée de faire un truc utile partout, tape juste une classe côté la hague et c'est tout.

Ok. Le patch qui va avec.

#19 Updated by Serghei Mihai about 1 year ago

div.block

c'est pour les pages d'authentic? On n'a pas la main sur ces cellules pour rajouter la classe, il me semble.

Tu géres aussi la taille du titre, ce n'est pas l'objet du ticket.

#20 Updated by Josué Kouka about 1 year ago

Serghei Mihai a écrit :

[...]
c'est pour les pages d'authentic? On n'a pas la main sur ces cellules pour rajouter la classe, il me semble.

Tu géres aussi la taille du titre, ce n'est pas l'objet du ticket.

Je suis censé le faire, vu que dans la patch qui enlève les bordures on a un $cell-title-cover-border: false;.

#21 Updated by Serghei Mihai about 1 year ago

Oui, et dans _cells.scss:

    @if $cell-title-cover-border == true {
        top: -1px;
        left: -1px;
        width: calc(100% + 2px);
    } else {
        width: 100%;
    }

J'ai appliqué ton patch en local mais je n'arrive pas à voir la différence entre les rendus avec le style de h2:first-child et sans.

#22 Updated by Josué Kouka about 1 year ago

  • File 0001-lahague-add-class-to-add-cell-border-24255.patch added

#23 Updated by Josué Kouka about 1 year ago

  • File deleted (0001-lahague-add-class-to-add-cell-border-24255.patch)

#25 Updated by Frédéric Péters about 1 year ago

Serghei écrivait, à propos de la partie div.block :

c'est pour les pages d'authentic? On n'a pas la main sur ces cellules pour rajouter la classe, il me semble.

Et le retrait de la partie pour les titres, c'est parce qu'au final Serghei n'a pas noté de différence ? Tu en voyais une, toi ?

#26 Updated by Josué Kouka 11 months ago

Je ne vois pas tant de différence mais cette section a une raison d'etre.

#27 Updated by Frédéric Péters 5 months ago

  • Status changed from Solution proposée to Rejeté

Si la demande revient, on verra.

Also available in: Atom PDF