search

Safaafeqri89 / ProyectoTFG

0 stars 0 forks source link

Revisión checkpoint 10 mayo #3

Open AVegMor opened 4 months ago

AVegMor commented 4 months ago

Hola.

Te falta el diario de trabajo. No tienes el código de la parte servidor.

Saludos

Safaafeqri89 commented 3 months ago

Hola buenas tardes

si lo tenia preparado pero al comprimirlo me daba error , voy a intentar subirlo Sino por lo menos en archivos separados Un saludo

De: AVegMor @.> Enviado: lunes, 13 de mayo de 2024 19:50 Para: Safaafeqri89/ProyectoTFG @.> Cc: Subscribed @.***> Asunto: [Safaafeqri89/ProyectoTFG] Revisión checkpoint 10 mayo (Issue #3)

Hola.

Te falta el diario de trabajo. No tienes el código de la parte servidor.

Saludos

— Reply to this email directly, view it on GitHubhttps://github.com/Safaafeqri89/ProyectoTFG/issues/3, or unsubscribehttps://github.com/notifications/unsubscribe-auth/A3CW4MTU25GJJS5ZC5AGLD3ZCD4OHAVCNFSM6AAAAABHUVEDT2VHI2DSMVQWIX3LMV43ASLTON2WKOZSGI4TGNBVGYZTEMQ. You are receiving this because you are subscribed to this thread.Message ID: @.***>

rlopdav392 commented 3 months ago

REVIEW PARTE FRONTEND

DISEÑO / FIGMA => Reusable (Atomic design) => ok => Responsive (responsive design) => No es responsive => interacciones => ok => Acuérdate que del diseño figma tiene que salir una plantilla de átomos que luego se actualizará en su maquetación CSS (https://drive.google.com/file/d/1Cn2LvjCbddrenFXy4i84zaV8F-1gjw5W/view) Colores => ok el color principal, pero tienes que citar también en la documentación que tienes un color acento, el naranja, en base a que regla cromática has seleccionado ese color acento, teniendo en cuenta tu color primario.

MAQUETACIÓN CSS3 => SASS,: no usas, si quieres puedes animarte a usarlo en el aplicativo mobile. => variables: perfecto => css nesting para hacerlo más legible (https://lenguajecss.com/postcss/plugins/css-nesting/)](https://lenguajecss.com/postcss/plugins/css-nesting/)](https://lenguajecss.com/postcss/plugins/css-nesting/): no, puedes usarlo con SASS en el aplicativo mobile. => Metodología BEM (Block--element__modifier): la usas en algunas clases (/logo--redes-sociales), pero no parece correcto, si ponemos un elemento, tienes que poner también el bloque del que viene, tiene que existir una clase logo padre. => Documentación interna: Algo más de comentarios, explicando lo que haces en las media querys. => Reusabilidad código - abstracción: Importante detallar en documentación / presentación nivel de reusabilidad de vuestra código (cuantas clases reusas en varios sitios, como los layouts (grid, flex), los botones, las listas, los espaciados (margin, padding),.., si haces uso de variables) => Arquitectura de ficheros: es OK => Big Layout / html semántico: tienes dentro del header otro header, además dentro del header deberías tener un nav, dentro del main quizás queda más limpio poner cada uno de los apartados principales como secciones, tienes un footer de cierre pero luego no veo un footer de apertura. => html semántico: todas las imágenes con su alt, has puesto title perfecto, y luego faltaría favicon y meta descricpción => Imágenes responsive (picture - source) y optimizadas (webp) => Responsive design: Importante detallar en documentación / presentación cuantos breakpoints se han definidos, esto es, para cuantas resoluciones es responsive vuestro diseño. => Plantilla de átomos: (https://drive.google.com/file/d/1Cn2LvjCbddrenFXy4i84zaV8F-1gjw5W/view) Acuédate que esta plantilla de átomos no es más que una actualización de la plantilla de átomos que salió de la maquetación CSS.

JAVASCRIPT => Información dinámica / estática: Dejar bien claro en la presentación / documentación que sección de tu página son dinámicas, se cargan desde servidor, y cuales estáticas. => Hacer el código lo más reusable posible haciendo uso de funciones e incluso de librerías/módulos. => Documentación interna / comentarios: es ok => DOM: delegación de eventos siempre que se pueda, y todos los eventos gestionados desde js (addeventlistener) y no desde html (onfocus = 'this.select()') => Arquitectura de ficheros: es ok

PENDIENTES => Terminar la parte javascript => Aplicativo mobile en react.

AVegMor commented 3 months ago

Hola Safaa,

he visto que has subido el vídeo entero. No hacía falta, podías haberlo subido a youtube o alguna plataforma similar y poner el enlace en el readme del proyecto. Por otro lado, te puedes bajar algún plugin del navegador y grabarte, yo suelo utilizar este https://chromewebstore.google.com/detail/nimbus-screenshot-screen/bpconcjcammlapcogcnnelfmaeghhagj?hl=es.

En cuanto a la parte servidor he visto que guardas los usuarios en BD y demás, pero no veo el código actualizado en github, ¿podrías subirlo?

AVegMor commented 3 months ago

Hola de nuevo, he visto que has subido código al repositorio de la parte de Laravel. Te he creado una pull request con algunos comentarios para que los tengas en cuenta. Además de lo que te he añadido en la pull request ten en cuenta lo siguiente:

Buen trabajo.