Projet

Général

Profil

Development #24255

la hague : classe pour avoir des bordures aux cellules

Ajouté par Frédéric Péters il y a presque 6 ans. Mis à jour il y a plus de 5 ans.

Statut:
Rejeté
Priorité:
Normal
Assigné à:
Josué Kouka
Version cible:
-
Début:
04 juin 2018
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:

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] »

Fichiers

0001-cells-add-no-border-class-24255.patch (1,15 ko) 0001-cells-add-no-border-class-24255.patch Josué Kouka, 04 juin 2018 14:19
no_boder.png (27,7 ko) no_boder.png Josué Kouka, 04 juin 2018 14:20
0002-lahague-revert-to-default-border-style-24255.patch (643 octets) 0002-lahague-revert-to-default-border-style-24255.patch Josué Kouka, 04 juin 2018 15:28
0001-cells-add-no-border-class-24255.patch (1,16 ko) 0001-cells-add-no-border-class-24255.patch Josué Kouka, 04 juin 2018 15:28
0001-cells-add-no-border-and-with-border-classes-24255.patch (1,57 ko) 0001-cells-add-no-border-and-with-border-classes-24255.patch Josué Kouka, 05 juin 2018 10:47
borders.png (20,6 ko) borders.png Josué Kouka, 05 juin 2018 10:49
0001-cells-add-with-border-class-24255.patch (1,36 ko) 0001-cells-add-with-border-class-24255.patch Josué Kouka, 05 juin 2018 18:00
0001-lahague-add-class-to-add-cell-border-24255.patch (704 octets) 0001-lahague-add-class-to-add-cell-border-24255.patch Josué Kouka, 06 juin 2018 14:41
lahague_cell_border.png (277 ko) lahague_cell_border.png Josué Kouka, 06 juin 2018 14:41
0001-lahague-add-class-to-add-cell-border-24255.patch (628 octets) 0001-lahague-add-class-to-add-cell-border-24255.patch Josué Kouka, 12 juin 2018 09:37
0001-lahague-add-class-to-add-cell-border-24255.patch (704 octets) 0001-lahague-add-class-to-add-cell-border-24255.patch Josué Kouka, 18 juillet 2018 11:12

Historique

#1

Mis à jour par Josué Kouka il y a presque 6 ans

  • Statut changé de Nouveau à En cours
#2

Mis à jour par Josué Kouka il y a presque 6 ans

#3

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

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

#5

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

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

#6

Mis à jour par Josué Kouka il y a presque 6 ans

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

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

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

Mis à jour par Josué Kouka il y a presque 6 ans

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

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

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

Mis à jour par Josué Kouka il y a presque 6 ans

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

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

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

#12

Mis à jour par Josué Kouka il y a presque 6 ans

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

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

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

#14

Mis à jour par Josué Kouka il y a presque 6 ans

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

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

Mais il n'y a plus de bordure sur les cellules depuis le commit dans Bug #23260: LaHague: retirer les bordures sur les bordures des cellules combo.

#16

Mis à jour par Josué Kouka il y a presque 6 ans

Serghei Mihai a écrit :

Mais il n'y a plus de bordure sur les cellules depuis le commit dans Bug #23260: LaHague: retirer les bordures sur les bordures des cellules combo.

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

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

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

#18

Mis à jour par Josué Kouka il y a presque 6 ans

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

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

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

Mis à jour par Josué Kouka il y a presque 6 ans

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

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

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

Mis à jour par Josué Kouka il y a presque 6 ans

  • Fichier 0001-lahague-add-class-to-add-cell-border-24255.patch ajouté
#23

Mis à jour par Josué Kouka il y a presque 6 ans

  • Fichier 0001-lahague-add-class-to-add-cell-border-24255.patch supprimé
#25

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

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

Mis à jour par Josué Kouka il y a presque 6 ans

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

#27

Mis à jour par Frédéric Péters il y a plus de 5 ans

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

Si la demande revient, on verra.

Formats disponibles : Atom PDF