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

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

Draft
lguerin wants to merge 8 commits from wip/79989-wcs-list-mode into main
pull from: wip/79989-wcs-list-mode
merge into: entrouvert:main
Conversation 2 Commits 8 Files Changed 10
lguerin commented 2023-07-27 15:35:16 +02:00
Owner
There is no content yet.
lguerin added 4 commits 2023-07-27 15:35:16 +02:00
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

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

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.
All checks were successful
gitea/combo/pipeline/head This commit looks good
Details
This pull request has changes conflicting with the target branch.
  • tests/wcs/test_card.py
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
There is no content yet.
Delete Branch "wip/79989-wcs-list-mode"

Deleting a branch is permanent. Although the deleted branch may exist for a short time before cleaning up, in most cases it CANNOT be undone. Continue?