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 8 mois. Mis à jour il y a 6 mois.

Statut:
Solution proposée
Priorité:
Normal
Assigné à:
Josué Kouka
Début:
04 juin 2018
Echéance:
% réalisé:

0%

Patch proposed:
Oui

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 Voir (1,15 ko) Josué Kouka, 04 juin 2018 14:19

no_boder.png Voir (27,7 ko) Josué Kouka, 04 juin 2018 14:20

0002-lahague-revert-to-default-border-style-24255.patch Voir (643 octets) Josué Kouka, 04 juin 2018 15:28

0001-cells-add-no-border-class-24255.patch Voir (1,16 ko) Josué Kouka, 04 juin 2018 15:28

0001-cells-add-no-border-and-with-border-classes-24255.patch Voir (1,57 ko) Josué Kouka, 05 juin 2018 10:47

borders.png Voir (20,6 ko) Josué Kouka, 05 juin 2018 10:49

0001-cells-add-with-border-class-24255.patch Voir (1,36 ko) Josué Kouka, 05 juin 2018 18:00

0001-lahague-add-class-to-add-cell-border-24255.patch Voir (704 octets) Josué Kouka, 06 juin 2018 14:41

lahague_cell_border.png Voir (277 ko) Josué Kouka, 06 juin 2018 14:41

0001-lahague-add-class-to-add-cell-border-24255.patch Voir (628 octets) Josué Kouka, 12 juin 2018 09:37

0001-lahague-add-class-to-add-cell-border-24255.patch Voir (704 octets) Josué Kouka, 18 juil. 2018 11:12

25176
25240
25288

Historique

#1 Mis à jour par Josué Kouka il y a 8 mois

  • Statut changé de Nouveau à En cours

#2 Mis à jour par Josué Kouka il y a 8 mois

25176

#3 Mis à jour par Frédéric Péters il y a 8 mois

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 8 mois

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 8 mois

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 8 mois

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 8 mois

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 8 mois

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 8 mois

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 Mis à jour par Serghei Mihai il y a 8 mois

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 8 mois

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 8 mois

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

#14 Mis à jour par Josué Kouka il y a 8 mois

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 8 mois

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

#16 Mis à jour par Josué Kouka il y a 8 mois

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 Mis à jour par Frédéric Péters il y a 8 mois

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

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 Mis à jour par Serghei Mihai il y a 7 mois

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

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

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

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

#23 Mis à jour par Josué Kouka il y a 7 mois

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

#24 Mis à jour par Josué Kouka il y a 7 mois

#25 Mis à jour par Frédéric Péters il y a 7 mois

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 6 mois

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

Formats disponibles : Atom PDF