mathieu/feature/gek Add updated InstructionView #412
Conversation
macteuts
commented
Oct 31, 2023
macteuts
commented
- 🍱 (DesignKit): Rename person.wave.2.svg to person_talking
- ✨ (GEK): Add updated InstructionView
There was a problem hiding this comment.
c'est du détail mais si tu peux changer le nom du svg :)
| struct ExerciseInstructionsButton: View { | ||
|
|
||
| @ObservedObject var viewModel: ActivityViewViewModel | ||
| @State private var synth = AVSpeechSynthesizer() |
There was a problem hiding this comment.
j'ai un bug avec le speech, ça marche pas et la console est flooded avec des outputs d'erreur
There was a problem hiding this comment.
Réglé mais comme on ne profite plus de la petite cuisine de SwiftUI il m'a fallu travailler avec le delegate de AVSpeechSynthesizer dans un ViewModel.
Détail important : sur simulateur il faut télécharger les 'voices' pour que ça fonctionne. De plus, le premier essai en 'build&run' sera toujours lent. Ce n'est plus le cas si vous relancez l'app sans logger.
|
|
||
| struct ExerciseInstructionsButton: View { | ||
|
|
||
| @ObservedObject var viewModel: ActivityViewViewModel |
There was a problem hiding this comment.
est-ce qu'on a besoin de garder tout le viewmodel?
est-ce qu'on peut pas juste utiliser le viewmodel pour en extraire le text
ou passer directement le text à la vue pour ne pas avoir de dépendance avec le view model?
ladislas
force-pushed
the
mathieu/feature/gek-Add-updated-InstructionView
branch
from
4b1f11f to
46cc761
Compare
ladislas
force-pushed
the
mathieu/feature/gek-Add-updated-InstructionView
branch
from
46cc761 to
da4f0dc
Compare
|
Kudos, SonarCloud Quality Gate passed!
|
