Dialogo final #308
Dialogo final #308
Conversation
There was a problem hiding this comment.
Quedó muy lindo el confeti 🚀 🚀
Lo que sí quizás se podría agregar la x para cerrar arriba a la derecha como tiene ahora en prod el dialogo:
Por las dudas que no se entienda del todo que hay que hacerle clic afuera para sacarlo.
Con el tema de las traducciones, no se @rgonzalezt si hay algun issue con migración de las traducciones al inglés y portugués, si no estaría bueno ya pasar las de el dialogo en este mismo PR.
| const EndDialog = () => | ||
| <Dialog | ||
| open={showModal} | ||
| disableRestoreFocus | ||
| fullWidth={true} | ||
| maxWidth="md" | ||
| onClose={() => setShowModal(false)}> | ||
| <DialogTitle id="draggable-dialog" sx={{ cursor: 'auto', fontWeight: 'bold', height: '50px', display: 'flex', alignItems: 'center' }}>{t('congratulationsModal.title')}</DialogTitle> | ||
| <DialogContent sx={{ overflow: "hidden", backgroundColor: theme.palette.background.default }}> | ||
| <Stack alignItems="center"> | ||
| <ConfettiExplosion {...{ force: 0.8, duration: 3000, particleCount: 250, width: 1600 }} /> | ||
| <img alt="register" width="25%" src="imagenes/primer-ciclo.png"></img> | ||
| <Stack display='flex' flexDirection='row' flexWrap='wrap' justifyContent='center' alignItems='center'> | ||
| <Typography style={{ display: 'inline-block', fontSize: '1.1rem', fontWeight: 'bold', fontStyle: 'italic' }}>{t("congratulationsModal.subtitle")} </Typography> | ||
| <Typography style={{ display: 'inline-block', fontSize: '1.1rem' }}>{t("congratulationsModal.text1")}</Typography> | ||
| </Stack> | ||
| <Typography style={{ fontSize: '1.1rem' }}>{t("congratulationsModal.text2")}</Typography> | ||
| </Stack> | ||
| </DialogContent> | ||
| </Dialog> |
There was a problem hiding this comment.
Esto podría estar en otro archivo, si bien la lógica de cuándo se tiene que mostrar tiene que estar acoplada al botón porque acá se hace la ejecución, toda la parte de componentes que hacen el dialogo y estilos (el componente EndDialog en sí) no tiene tanto que ver con la ejecución asi que mejor que esté por separado.
|
No hay un issue de traducciones, traduzcamos a medida que vamos creando nuevos textos! |
|
|
@rgonzalezt @dlopezalvas si les parece con los cambios que pasé recien, me avisan y hago el merge ! |
|
Copien traducciones existentes ya en ember!! Gran laburo gente linda!! |
|
quedó buenísimo!! |
Resolves #253
enddialog.mp4