KenrouIt / cheddar-ecosystem

0 stars 1 forks source link

Header improved #45

Closed Mart-Dan-Rossi closed 2 months ago

eliasgreen18 commented 3 months ago

Adjunto evidencia de lo que se ve al cargar la prueba.

Captura de pantalla 2024-06-06 a la(s) 00 29 43

Captura de pantalla 2024-06-06 a la(s) 00 29 48

El menú desplegable aparece con ese texto raro long...tnet y al desplegar sí se observan correctamente las opciones del balance y logOut en caso de estar ya conectado. ¿El long...tnet es el comportamiento esperado?

Revisando la salida de backend veo que eso hace referencia a longhost7912.testnet que queda acotado para entrar en el desplegable.

Captura de pantalla 2024-06-06 a la(s) 00 29 35

Captura de pantalla 2024-06-06 a la(s) 00 29 28

Cuando utilicé la opción de logOut pero finalmente cerré la pestaña que se abrió automáticamente se presentó el error que se observa en la captura.

Captura de pantalla 2024-06-06 a la(s) 00 29 56

El archivo MP4 cargó correctamente y se puede reproducir adecuadamente en el modo escritorio.

eliasgreen18 commented 3 months ago

Si estamos OK con eso, lo veo bien funcionalmente

Mart-Dan-Rossi commented 3 months ago

Adjunto evidencia de lo que se ve al cargar la prueba.

Captura de pantalla 2024-06-06 a la(s) 00 29 43

Captura de pantalla 2024-06-06 a la(s) 00 29 48

El menú desplegable aparece con ese texto raro long...tnet y al desplegar sí se observan correctamente las opciones del balance y logOut en caso de estar ya conectado. ¿El long...tnet es el comportamiento esperado?

Captura de pantalla 2024-06-06 a la(s) 00 29 35

Captura de pantalla 2024-06-06 a la(s) 00 29 28

Cuando utilicé la opción de logOut pero finalmente cerré la pestaña que se abrió automáticamente se presentó el error que se observa en la captura.

Captura de pantalla 2024-06-06 a la(s) 00 29 56

El archivo MP4 cargó correctamente y se puede reproducir adecuadamente en el modo escritorio.

El texto raro es la cuenta logueada mostrando los primeros y los últimos caracteres de la misma. Yo había pensado q aparecieran la misma cantidad de caracteres pero con los puntos suspensivos en el final. Dani me pidió que lo cambie para que quede de esta forma. De todos modos tal vez podríamos agregarle un "title" o algo así para q al hacerle hover aparezca el nombre completo pero de momento es lo pedido.

El error, si no me equivoco, lo salva Ro en otro branch. Faltaría mergear ambos y ver q quede bien (Porfa @RodrigoCSolari confirmame esto).

Lo del MP4, pos más vale pa.. si soy un maestro (?) jajja <3

RodrigoCSolari commented 3 months ago

confirmado el error fue resuelto en el PR 47, el cual fue aprobado y mergeado a dev

Mart-Dan-Rossi commented 2 months ago

Ok, i think specifications have changed a lot since the PR was created. Last indications were following this desing : Mobile: image

Desktop: image


Considering those indication burguer button and responsiveness were added.

In some point i had orders to add a music button for the cheddar rap, in others i had instructions to make it open when clicking in the cheddar icon. I added a music icon in a button.

Scoreboard button is mocked. Functionallity should be added when mergin with the implement-scoreboard branch.

In regard of the missing nav buttons/links, i still dindn't get any of the nav links so i didn't add em yet.

NOTE: I accept styles changes if you want to change something.

Kenrou7 commented 2 months ago

[mountain] Por lo que veo, en este instante, el PR tiene conflictos. Recientemente aprobé el PR de scoreboard, con lo que puede estar relacionado. Por lo que leo del PR, se menciona también que se esperaba a que se aprobara ese PR, con lo que ya estamos en condiciones de hacer un pull desde dev y asegurarse de implementar bien eso

Mart-Dan-Rossi commented 2 months ago

[mountain] Por lo que veo, en este instante, el PR tiene conflictos. Recientemente aprobé el PR de scoreboard, con lo que puede estar relacionado. Por lo que leo del PR, se menciona también que se esperaba a que se aprobara ese PR, con lo que ya estamos en condiciones de hacer un pull desde dev y asegurarse de implementar bien eso

Pull de dev hecho. Funcionalidad de scoreboard button incorporada exitosamente.