entrouvert
/
combo
16
1
Fork 0
Code Issues Pull Requests 11 Releases Wiki Activity

cellule fiche(s): avoir un display mode "Liste" (#79989) #146

Merged
lguerin merged 7 commits from wip/79989-wcs-list-mode into main 2023-12-20 08:26:24 +01:00
Conversation 6 Commits 7 Files Changed 10 +1213 -664
lguerin commented 2023-07-27 15:35:16 +02:00
Owner
Copy Link
No description provided.
lguerin changed title from cellule fiche(s): avoir un display mode "Liste" (#79989) to WIP: cellule fiche(s): avoir un display mode "Liste" (#79989) 2023-07-27 15:35:26 +02:00
tjund requested review from tjund 2023-07-27 15:52:56 +02:00
lguerin force-pushed wip/79989-wcs-list-mode from 5a9635849a to 9e5dfae560 2023-08-02 11:39:46 +02:00 Compare
lguerin changed title from WIP: cellule fiche(s): avoir un display mode "Liste" (#79989) to WIP: cellule fiche(s): avoir un display mode "Liste" (#79989) 2023-08-02 11:48:39 +02:00
lguerin changed target branch from wip/79486-wcs-card-info to main 2023-08-02 11:48:40 +02:00
lguerin force-pushed wip/79989-wcs-list-mode from 9e5dfae560 to d085b07004 2023-08-02 11:51:55 +02:00 Compare
lguerin force-pushed wip/79989-wcs-list-mode from d085b07004 to 42119e7eeb 2023-08-02 12:09:33 +02:00 Compare
tjund added 3 commits 2023-09-04 09:11:55 +02:00
tjund commented 2023-09-04 09:34:04 +02:00
Owner
Copy Link

Ajouter 2 commits.

d8beacc349
Pour éviter que new Card_cell_custom() soit lancer 3× pour chaque cellule fiche.

f0d524d559
Permet de stocker un schéma différent pour chaque mode d'affichage. Cela permet à l'usager de modifier les champs du mode tableau sans bouger celui du mode card ou liste.
Par contre, seul le shéma du mode d'affichage choisi sera sauvegardé, ce qui fait qu'en cas de sauvegarde de la cellule, seul le shéma du mode actif est sauvegardé. À voir si cela est gênant ou si sauvegarder le dict contenant les 3 schéma ne serait pas plus interessant d'un point du vue UI.
Ce commit ouvre la voie à la possibilité de simplifier l'interface en mode liste

Ajouter 2 commits. https://git.entrouvert.org/entrouvert/combo/commit/d8beacc349594e4f4f4ff5d13187870d38ff8e1c Pour éviter que `new Card_cell_custom()` soit lancer 3× pour chaque cellule fiche. https://git.entrouvert.org/entrouvert/combo/commit/f0d524d559d4fb0298aa30642c446fd8658151cf Permet de stocker un schéma différent pour chaque mode d'affichage. Cela permet à l'usager de modifier les champs du mode tableau sans bouger celui du mode card ou liste. Par contre, seul le shéma du mode d'affichage choisi sera sauvegardé, ce qui fait qu'en cas de sauvegarde de la cellule, seul le shéma du mode actif est sauvegardé. À voir si cela est gênant ou si sauvegarder le dict contenant les 3 schéma ne serait pas plus interessant d'un point du vue UI. Ce commit ouvre la voie à la possibilité de simplifier l'interface en mode liste
tjund added 1 commit 2023-09-06 10:59:58 +02:00
tjund commented 2023-09-06 11:09:24 +02:00
Owner
Copy Link

et enfin
9e4d77e371

Qui simplifie l'interface du mode liste en ne proposant que la possibilité de modifier un élément de type @link@.

Test bienvenu.

(d'autres modifs seraient appréciables, je note ça pour partager mes envies de refacto, comme renommer certains noms de variables : grid n'a plus aucun sens, séparer plus proprement côté JS les modes card, table et list, pourquoi pas dans des objets séparés pour éviter cet enfer d’if/else et vraiment simplifier grid_cell__set_schema et grid_cell__edit_set_fields : c'est illisible.

et enfin https://git.entrouvert.org/entrouvert/combo/commit/9e4d77e3716fc3a3475079719c2f943ad9e8a95a Qui simplifie l'interface du mode liste en ne proposant que la possibilité de modifier un élément de type @link@. Test bienvenu. (d'autres modifs seraient appréciables, je note ça pour partager mes envies de refacto, comme renommer certains noms de variables : `grid` n'a plus aucun sens, séparer plus proprement côté JS les modes card, table et list, pourquoi pas dans des objets séparés pour éviter cet enfer d’if/else et vraiment simplifier grid_cell__set_schema et grid_cell__edit_set_fields : c'est illisible.
lguerin force-pushed wip/79989-wcs-list-mode from 9e4d77e371 to 070ebe913a 2023-11-13 16:07:26 +01:00 Compare
lguerin added 1 commit 2023-11-13 16:11:11 +01:00
gitea/combo/pipeline/head There was a failure building this commit Details
ea430f6528
fixup! wcs: add specific templates for display mode list (#79989)
lguerin commented 2023-11-13 16:21:05 +01:00
Author
Owner
Copy Link

J'ai rebasé la branche, réglé les conflits, et ajouté un fixup sur ton dernier commit.

En testant un peu, je remarque que lorsqu'on édite une cellule d'une custo, et qu'on change le type de contenu (exemple, passage d'un champ de la fiche à un champ utilisateur), le select suivant n'est pas automatiquement positionné sur la première valeur possible (c'est le cas dans main), le select apparaît vide.

J'ai rebasé la branche, réglé les conflits, et ajouté un fixup sur ton dernier commit. En testant un peu, je remarque que lorsqu'on édite une cellule d'une custo, et qu'on change le type de contenu (exemple, passage d'un champ de la fiche à un champ utilisateur), le select suivant n'est pas automatiquement positionné sur la première valeur possible (c'est le cas dans main), le select apparaît vide.
lguerin force-pushed wip/79989-wcs-list-mode from ea430f6528 to de0d7efe14 2023-11-13 16:24:32 +01:00 Compare
tjund commented 2023-12-06 15:21:27 +01:00
Owner
Copy Link

Je reproduit exactement le même bug sur main. SI tu confirmes @lguerin , je propose d'isoler ce bug sur un autre ticket, vu qu'il n'y a pas régression et passer cette évolution en l'état.

Je reproduit exactement le même bug sur main. SI tu confirmes @lguerin , je propose d'isoler ce bug sur un autre ticket, vu qu'il n'y a pas régression et passer cette évolution en l'état.
tjund commented 2023-12-06 15:52:19 +01:00
Owner
Copy Link

Je pense que ça provient du commit de juillet avec l'ajout de

grid_cell__edit_set_fields: function(grid_cell) {
    for (let field of this.grid_cell_form.elements) {
      field.value = '';
    }

Et je serait bien tenté du coup de reprendre cette methode pour vraiment essayer de la simplifier.

Je pense que ça provient du commit de juillet avec l'ajout de ``` JS grid_cell__edit_set_fields: function(grid_cell) { for (let field of this.grid_cell_form.elements) { field.value = ''; } ``` Et je serait bien tenté du coup de reprendre cette methode pour vraiment essayer de la simplifier.
lguerin commented 2023-12-07 09:49:09 +01:00
Author
Owner
Copy Link

ok pour régler le soucis dans un autre ticket

ok pour régler le soucis dans un autre ticket
lguerin changed title from WIP: cellule fiche(s): avoir un display mode "Liste" (#79989) to cellule fiche(s): avoir un display mode "Liste" (#79989) 2023-12-07 09:49:12 +01:00
lguerin force-pushed wip/79989-wcs-list-mode from de0d7efe14 to 60f012ca1a 2023-12-07 09:49:58 +01:00 Compare
tjund approved these changes 2023-12-19 14:35:59 +01:00
lguerin force-pushed wip/79989-wcs-list-mode from 60f012ca1a to b02801ebc8 2023-12-19 14:37:15 +01:00 Compare
lguerin merged commit b02801ebc8 into main 2023-12-20 08:26:24 +01:00
lguerin deleted branch wip/79989-wcs-list-mode 2023-12-20 08:26:24 +01:00
Sign in to join this conversation.
Reviewers
No reviewers
tjund
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: entrouvert/combo#146
No description provided.
Delete Branch "wip/79989-wcs-list-mode"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?