Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

COM-3842: add slot in modal attendance sheet #2086

Merged
merged 7 commits into from
Nov 27, 2024
Merged

COM-3842: add slot in modal attendance sheet #2086

merged 7 commits into from
Nov 27, 2024

Conversation

Dezau1995
Copy link
Collaborator

  • J'ai vérifié la fonctionnalité sur mobile
  • J'ai ajouté une variable d'environnement

Si tu as lu cette description, pense à réagir avec un 👁️

@Dezau1995 Dezau1995 self-assigned this Nov 21, 2024
@manonpalin
Copy link
Collaborator

manonpalin commented Nov 22, 2024

J'ai repéré un soucis lorsque pour une formation individuelles, tous les creneaux sont rattachés a une feuille d'émargement, on peut encore cliquer sur le bouton pour ajouter une feuille d'emargement, :

Enregistrement.de.l.ecran.2024-11-22.a.14.59.17.mov

je pense qu'il faudrait bloquer ce comportement en ajoutant la condition (!isSingleCourse || notLinkedSlotOptions.length) sur le bouton Ajouter une feuille d'emargement avec isSingleCourse qui vient de la composable AttendanceSheets

@manonpalin
Copy link
Collaborator

manonpalin commented Nov 26, 2024

C'est un peu du détail mais ca me perturbe que ca soit pas centré ^^ Que penses-tu de centrer l'affichage des options quand inline vaut true ?
Description de l'image

Pour ca, tu peux ajouter dasn OptionGroup : :class="['q-px-sm', {'options': inline}]"
avec
.options
display: flex
flex-wrap: wrap
justify-content: space-around ou space-between

SI tu fais la modif il faudrait tester qu'on ne casse pas laffichage des option-group ailleurs dans l'app (quand on genere un ordre de mission depuis la fiche d'un formateur, quand on defini les certifications d'une formation etc je te laisserai faire la liste ;) )

@ulysseferreira ulysseferreira merged commit 6064d1a into dev Nov 27, 2024
3 checks passed
@ulysseferreira ulysseferreira deleted the COM-3842 branch November 27, 2024 16:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants