entrouvert
/
chrono
14
1
Fork 0
Code Issues Pull Requests 5 Releases Wiki Activity

plages libres résa occasionnelle (#80050) #127

Merged
vdeniaud merged 3 commits from wip/80050-plages-libres-permettre-la-reser into main 2023-09-19 17:16:36 +02:00
Conversation 2 Commits 3 Files Changed 9 +215 -47
vdeniaud commented 2023-08-09 15:43:39 +02:00
Owner
Copy Link
No description provided.
vdeniaud force-pushed wip/80050-plages-libres-permettre-la-reser from ab624678ac to 5ffaedbdfc 2023-08-09 16:02:18 +02:00 Compare
tjund commented 2023-09-07 11:01:00 +02:00
Owner
Copy Link

J'ai essayé côté CSS de mettre plusieurs barres sur une seule ligne. Commit poussé.
Idée retenue : la première barre est en position relative, pour rester dans le flux et donner une hauteur au container, les suivantes sont en absolute.
Ce n'est pas parfait, un souci peut se produire lorsqu'une barre (si elle n'est pas la première) passe sur 2 lignes, la hauteur du container ne suit pas.
Les autres alternatives ne sont pas forcément mieux, je laisse comme ça pour le moment, je ne me rends pas compte si ce bug se produira souvent ou jamais. À voir à l'usage.
Sinon, je note ici une solution la plus simple pour régler le problème graphiquement, qui serait de modifier le tri des {{user.booking}} pour faire passer le plus court en premier {{ user.bookings|dictsort="css_width" }}, mais comme on perd un ordre logique de liste des périodes dans le code. je n'ai pas appliqué.

J'ai essayé côté CSS de mettre plusieurs barres sur une seule ligne. Commit poussé. Idée retenue : la première barre est en position relative, pour rester dans le flux et donner une hauteur au container, les suivantes sont en absolute. Ce n'est pas parfait, un souci peut se produire lorsqu'une barre (si elle n'est pas la première) passe sur 2 lignes, la hauteur du container ne suit pas. Les autres alternatives ne sont pas forcément mieux, je laisse comme ça pour le moment, je ne me rends pas compte si ce bug se produira souvent ou jamais. À voir à l'usage. Sinon, je note ici une solution la plus simple pour régler le problème graphiquement, qui serait de modifier le tri des {{user.booking}} pour faire passer le plus court en premier `{{ user.bookings|dictsort="css_width" }}`, mais comme on perd un ordre logique de liste des périodes dans le code. je n'ai pas appliqué.
vdeniaud force-pushed wip/80050-plages-libres-permettre-la-reser from ae92530b46 to f342ea2c0e 2023-09-14 11:56:10 +02:00 Compare
vdeniaud commented 2023-09-14 11:58:28 +02:00
Author
Owner
Copy Link

Ce n'est pas parfait, un souci peut se produire lorsqu'une barre (si elle n'est pas la première) passe sur 2 lignes, la hauteur du container ne suit pas.

Je vais ouvrir un ticket pour garder trace de ce problème, on le traitera plus tard, quand l'interface sera davantage stabilisée.

Et pour le problème dont je parle dans le ticket (les API qui se marchent dessus à cause du même user_external_id), autre ticket également.

> Ce n'est pas parfait, un souci peut se produire lorsqu'une barre (si elle n'est pas la première) passe sur 2 lignes, la hauteur du container ne suit pas. Je vais ouvrir un ticket pour garder trace de ce problème, on le traitera plus tard, quand l'interface sera davantage stabilisée. Et pour le problème dont je parle dans le ticket (les API qui se marchent dessus à cause du même user_external_id), autre ticket également.
vdeniaud changed title from WIP: plages libres résa occasionnelle (#80050) to plages libres résa occasionnelle (#80050) 2023-09-14 12:50:15 +02:00
lguerin approved these changes 2023-09-15 09:20:47 +02:00
vdeniaud force-pushed wip/80050-plages-libres-permettre-la-reser from f342ea2c0e to 8b924ef670 2023-09-19 17:01:23 +02:00 Compare
vdeniaud merged commit 8b924ef670 into main 2023-09-19 17:16:36 +02:00
vdeniaud deleted branch wip/80050-plages-libres-permettre-la-reser 2023-09-19 17:16:36 +02:00
Sign in to join this conversation.
Reviewers
No reviewers
lguerin
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
3 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/chrono#127
No description provided.
Delete Branch "wip/80050-plages-libres-permettre-la-reser"

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?