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

partial-bookings: update manager html & css (#78728) #104

Merged
vdeniaud merged 1 commits from wip/78728-manager-vue into main 2023-07-03 17:41:36 +02:00
Conversation 4 Commits 1 Files Changed 4 +144 -49
tjund commented 2023-06-21 17:46:18 +02:00
Owner
Copy Link
No description provided.
tjund added 1 commit 2023-06-21 17:46:19 +02:00
tjund force-pushed wip/78728-manager-vue from 76228b65b5 to 2cd7dacd43 2023-06-21 17:52:05 +02:00 Compare
tjund changed title from partial-bookings: update manager html & css (#78728) to WIP: partial-bookings: update manager html & css (#78728) 2023-06-22 09:51:56 +02:00
vdeniaud commented 2023-06-22 13:57:06 +02:00
Owner
Copy Link

Ça me paraît aller tout à fait dans la bonne direction, il faudra juste que je fasse un tour sur le html le moment venu.

Suggestions d'améliorations :

  • Les lignes blanches paraissent plus hautes que les lignes grises.
  • Une réservation qui se finit à 16h30 n'a pas sa délimitation finale au milieu d'une case, elle s'arrête un peu avant.
  • Les heures pourraient être centrées au dessus du trait qui délimite deux case, plutôt qu'après le trait ?
Ça me paraît aller tout à fait dans la bonne direction, il faudra juste que je fasse un tour sur le html le moment venu. Suggestions d'améliorations : * Les lignes blanches paraissent plus hautes que les lignes grises. * Une réservation qui se finit à 16h30 n'a pas sa délimitation finale au milieu d'une case, elle s'arrête un peu avant. * Les heures pourraient être centrées au dessus du trait qui délimite deux case, plutôt qu'après le trait ?
1687434796.png
19 KiB
1687434796.png
tjund force-pushed wip/78728-manager-vue from 2cd7dacd43 to afe06bb389 2023-06-28 18:13:24 +02:00 Compare
tjund commented 2023-06-28 18:20:06 +02:00
Author
Owner
Copy Link

Quelques améliorations CSS:

  • suppression du margin entre les participants pour egaliser visuellement les lignes blanches et grises.
  • Amélioration de la vue mobile.
  • Suppression de la référence de la première heure.
  • Centrage des heures.
  • Utilisation de la balise

Une réservation qui se finit à 16h30 n'a pas sa délimitation finale au milieu d'une case, elle s'arrête un peu avant.

La faute au calcul de position en pourcentage qui retourne un entier. Il faut que les valeurs left et width soient retournées avec une decimale pour avoir un positionnement plus précis.

Quelques améliorations CSS: * suppression du margin entre les participants pour egaliser visuellement les lignes blanches et grises. * Amélioration de la vue mobile. * Suppression de la référence de la première heure. * Centrage des heures. * Utilisation de la balise <time> pour déclarer le heures (a11y). > Une réservation qui se finit à 16h30 n'a pas sa délimitation finale au milieu d'une case, elle s'arrête un peu avant. La faute au calcul de position en pourcentage qui retourne un entier. Il faut que les valeurs left et width soient retournées avec une decimale pour avoir un positionnement plus précis.
tjund commented 2023-06-29 10:34:29 +02:00
Author
Owner
Copy Link

Concernant le support de position: sticky (le « Très chouette les heures qui restent en haut quand on scrolle »).
Il n'est pas supporter dans gadjo ou plutôt désactivé par la propriété overflow posée globalement sur #main-content.
J'ai déjà proposé de la supprimer via #62142, mais ce fût une mauvaise idée, restaurée par #62249. J'ai pas encore les idées claires là-dessus. Peut-être une class #main-content.no-overflow qui permettrait de désactiver l'overflow progressivement, mais je préfère en faire un ticket dédié.

je retire le WIP.

Concernant le support de `position: sticky` (le « Très chouette les heures qui restent en haut quand on scrolle »). Il n'est pas supporter dans gadjo ou plutôt désactivé par la propriété `overflow` posée globalement sur #main-content. J'ai déjà proposé de la supprimer via #62142, mais ce fût une mauvaise idée, restaurée par #62249. J'ai pas encore les idées claires là-dessus. Peut-être une class #main-content.no-overflow qui permettrait de désactiver l'overflow progressivement, mais je préfère en faire un ticket dédié. je retire le WIP.
tjund changed title from WIP: partial-bookings: update manager html & css (#78728) to partial-bookings: update manager html & css (#78728) 2023-06-29 10:34:40 +02:00
vdeniaud commented 2023-06-29 16:11:45 +02:00
Owner
Copy Link

Super top !

Concernant le support de position: sticky
je préfère en faire un ticket dédié.

Yep c'est loin d'être indispensable, ne bloquons pas là dessus.

La faute au calcul de position en pourcentage qui retourne un entier. Il faut que les valeurs left et width soient retournées avec une decimale pour avoir un positionnement plus précis.

C'est fait dans la branche et effectivement tout est bien positionné désormais, par contre ça introduit un mini décalage pour une réservation qui démarre sur une heure pile, cf screenshot. À toi de me dire si ça se corrige ou si on néglige.

@csechet je ne suis pas compétent pour relire le CSS, tu peux prendre ?

Super top ! > Concernant le support de `position: sticky` > je préfère en faire un ticket dédié. Yep c'est loin d'être indispensable, ne bloquons pas là dessus. > La faute au calcul de position en pourcentage qui retourne un entier. Il faut que les valeurs left et width soient retournées avec une decimale pour avoir un positionnement plus précis. C'est fait dans la branche et effectivement tout est bien positionné désormais, par contre ça introduit un mini décalage pour une réservation qui démarre sur une heure pile, cf screenshot. À toi de me dire si ça se corrige ou si on néglige. @csechet je ne suis pas compétent pour relire le CSS, tu peux prendre ?
1688047837.png
3.6 KiB
1688047837.png
csechet requested review from csechet 2023-06-30 08:59:10 +02:00
csechet approved these changes 2023-06-30 10:00:12 +02:00
csechet left a comment
Owner
Copy Link

Juste une typo dans un nom de variable

Juste une typo dans un nom de variable
chrono/manager/static/css/style.scss Outdated
@ -604,1 +601,3 @@
outline: solid 1px;
// Partial booking view
.partial-booking {
--registant-name-width: 15rem;
csechet commented 2023-06-30 09:55:02 +02:00
Owner
Copy Link

registant => registrant

registant => registrant
vdeniaud force-pushed wip/78728-manager-vue from 9b0fd092ea to 51f34dc143 2023-07-03 17:01:55 +02:00 Compare
vdeniaud force-pushed wip/78728-manager-vue from 51f34dc143 to 00ad7b0747 2023-07-03 17:29:21 +02:00 Compare
vdeniaud merged commit 00ad7b0747 into main 2023-07-03 17:41:36 +02:00
vdeniaud deleted branch wip/78728-manager-vue 2023-07-03 17:41:36 +02:00
Sign in to join this conversation.
Reviewers
No reviewers
csechet
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#104
No description provided.
Delete Branch "wip/78728-manager-vue"

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?