Skip to content

Refactor Botonera de la Escena#336

Merged
rgonzalezt merged 13 commits intodevelopfrom
refactorSceneButtons
Sep 16, 2025
Merged

Refactor Botonera de la Escena#336
rgonzalezt merged 13 commits intodevelopfrom
refactorSceneButtons

Conversation

@danielferro69
Copy link
Contributor

Refactor #335

@danielferro69 danielferro69 requested a review from a team as a code owner August 13, 2025 13:53
Copy link
Contributor

@rgonzalezt rgonzalezt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Comparado con el comportamiento de actual, me dí cuenta que faltaba mantener el estado del botón de reinicio cuando se termina de ejecutar. Propuse una solución acá: #338

Si te parece habría que mergear ese pr en esta branch.

Lo que faltaría ver es replicar esta lógica de mantener el estado del botón de reinicio en el step by step.

const handleRestart = () => {
scene.restartScene(challenge.sceneDescriptor);
setRunning && setRunning(false);
setInterpreterVersion(v => v + 1);
Copy link
Contributor

@rgonzalezt rgonzalezt Aug 20, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

no entendí para qué es lo de la versión 🤔

@danielferro69
Copy link
Contributor Author

En este ultimo commit, resolvemos lo mencionado en el PR 338 sobre el estado de los botones segun termina la ejecucion con el boton reiniciar y solucionamos el tema de los colores hover sobre el boton paso a paso y el color cuando ese mismo boton en modo vertical esta deshabilitado

Copy link
Contributor

@rgonzalezt rgonzalezt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Súper ultra regio!!!! 🚀

@rgonzalezt rgonzalezt merged commit d7e1ad0 into develop Sep 16, 2025
10 checks passed
@rgonzalezt rgonzalezt deleted the refactorSceneButtons branch September 16, 2025 17:36
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.

2 participants

Comments