-
Notifications
You must be signed in to change notification settings - Fork 29
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
Chuy haciendo jueguito #1477
Chuy haciendo jueguito #1477
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Deje algunos comentarios abajo.
Sumado a lo otro, creo que queda medio raro que revolear la pelota sea la animación de la pelota de basquet al ser la misma que estando en reposo, da la sensación de que no hizo algo.
app/utils/staticData/desafios.js
Outdated
id: 'nuevo2', | ||
nombre: 'ChuyHaciendoJueguito', | ||
escena: 'ChuyHaciendoJueguito', | ||
bloques: ['Avanzar', 'Volver', 'Retroceder', 'RecogerPulpito', 'RebotarPiePulpito', 'RevolearPulpito', 'Repetir', 'Procedimiento'], |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
¿Puede ser que no sea necesario el Volver/Retroceder? Si hacen cosas diferentes, creo que sería mejor que tengan diferentes iconos porque me dio la sensación de que hacen lo mismo.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
En realidad los bloques volver y retroceder ya existian y los estoy usando. Como que el ejercicio termina con el bloque volver. El avanzar y retroceder los uso para que entre en calor...
Asi lo pedia el issue, pero lo podemos analizar.
|
||
const nombre = "ChuyHaciendoJueguito"; | ||
|
||
moduloActividad(nombre, () =>{ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Por ahí estaría bueno tener testeadas diferentes soluciones con diferente orden. Probé y vi que se puede rebotar la pelota antes de agarrarla, idem con revolearla. Creo que eso habría que ver en los estados en exercises, pero a la vez estaría bueno que hayan tests de esos casos.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
dale, veo de agregar un test con error.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ahi agregué el test con error.
@@ -51,6 +52,9 @@ blowUpBallon: Pinch the balloon | |||
kickBall: Kick the ball | |||
kickFootballBall: Kick the football ball | |||
bounceRubberBall: Bounce the rubber ball | |||
volleyRubberBall: Volley the rubber ball |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Me suena rara la traducción 'volley', ¿podría ser throw the rubber ball up
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
el ingles, se los dejo... la realidad es que revolear pelota en ingles es precisamente volley ball jajajaj ! pero podemos poner que sea tirar la pelota para arriba... no hay problema.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Jajajajajj mirá vos, claro, volley.
this.crearBloqueAccion('RebotarUnaVezPingPong', 'bouncePingPong', { | ||
icono: 'icono.paleta.png', | ||
comportamiento: 'PingPongAnimado', | ||
argumentos: '{mensaje: "Rebotando...", etiqueta: "PingPong", nombreAnimacion:"usarPaletaUnaVez", animacionInteractuadoMientras: "desaparecer" }', | ||
}); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No se llega a ver la cantidad de veces que rebotó, queda siempre el mensaje Rebotando...
, ¿se puede usar el mismo comportamiento que tiene el ejercicio original? SaltarHablando
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Revisar https://github.com/Program-AR/pilas-bloques-exercises/blob/master/src/comportamientos/SaltarHablando.ts y modificar el personaje si es necesario
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@dlopezalvas @asanzo Respecto al tema de los rebotes, hice un cambio en exercises porque este comportamiento de SaltarHablando es para el actor, en ese caso, el Gato (GatoAnimado). Y lo que hace es mover hacia arriba al Gato con una velocidad inicial, una altura, velocidad y cantidad de pasos. En este caso con Chuy es distinto, porque la animacion es la de rebotar la pelota de ping pong con la paleta que es una animacion propia de Chuy y no es Chuy saltando o rebotando. Entonces, lo que hice fue modificar el mensaje antes de crear el globo en la preAnimacion del comportamiento PingPongAnimado y de esa forma, en lugar de ver la palabra Rebotando... se ve la cantidad que lleva de rebotes. Me pareció que esta era una buena solucion.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Genial, yo aprovecharía la resolución de conflictos para hacer bump de la versión de exercises.
Resolves #1355
nuevo2.mp4
nuevo3.mp4
PBE Program-AR/pilas-bloques-exercises#52