search

SantiTru / FLEKI-BOLAS_PADEL_TEAM

Página web equipo profesional de pádel
0 stars 1 forks source link

Revisión checkpoint 10 mayo #1

Open AVegMor opened 4 months ago

AVegMor commented 4 months ago

Hola.

No tienes el readme actualizado con todas las tareas que habéis ido haciendo, aparecen muy poquitas. Os falta el video enlazado. No veo el código de la parte servidor.

Saludos

SantiTru commented 4 months ago

Buenas!

Estamos trabajando en completar lo que nos falta. En cuanto avancemos te lo hago saber. Un saludo!

rlopdav392 commented 3 months ago

Tema frontend: He visto que habéis dejado lo del espaciado en el calendario, yo he investigado y no encuentro solución: https://stackoverflow.com/questions/30501015/z-index-not-working-inside-scroll-box Explican, que el modal al tener como container padre al calendario el cual tiene asignado el scroll, pues que no es posible, que la solución que plantean, es sacar al modal fuera del container padre que tiene el scroll. No se si habrá otra forma, pero eso es lo que he encontrado a priori. Así que si os parece, lo dejamos así, con ese espacio y ya. Pero una cosilla, ese calendario es estático y tiene puestos los eventos desde el html y no desde el javascript con una delegación de eventos, no se si habíais pensando luego hacerlo dinámico, o de momento que sea estático, ya que preferis priorizar otro contenido como dinámico, ya me diréis. En caso de que se vaya a quedar estático, si que si os da tiempo, estaría genial que los eventos se gestionaran todos desde js y no desde html, con una delegación de eventos.

rlopdav392 commented 3 months ago

ENTONCES, PEQUEÑO RESUMEN REVIEW PARTE FRONTEND:

DISEÑO / FIGMA Está perfecto Reusable (Atomic design) => ok Responsive (responsive design) => ok interacciones => ok Acordaros 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) Y acordaros que hay que pasarle algún plugin de accesibilidad Y explicar en la documentación: patrones de diseño y principios de diseño

MAQUETACIÓN CSS3 => Vosotros no habéis usado SASS, pero aunque no hagáis uso de SASS podéis usar variables para hacerlo más reusable, y anidamientos (css nesting) para hacer lo más legible: (https://lenguajecss.com/postcss/plugins/css-nesting/) => También si os animáis podéis hacer uso de la metodología BEM, y si os diese tiempo. => Comentarios: ok => código - reusabilidad - abstracción: acordaros que cuantas más clases abstractas como la de u-margin-bottom-small, pues más reusable. => Arquitectura de ficheros CSS: es ok, podéis incluso dividirlos más: un fichero general.css con las configuraciones globales, los componetes reusables y las clases reusables, otro para las querys y otro para la página en si. => Big layout / html semántico: la página principal tiene que tener un header con su nav, un main con sus section y un footer. => SEO: meta descripción, todas las imágenes con su alt, favicon => Imágenes responsive 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: ya he visto que ya tenéis una boceto de plantilla de átomos en un css, si os da tiempo completarla un poco más: (https://drive.google.com/file/d/1Cn2LvjCbddrenFXy4i84zaV8F-1gjw5W/view) Acordaros 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 vuestra 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 (onClick) => Arquitectura de ficheros: es ok, lo único cambiar el nombre script.js por uno más claro, por ejemplo index.js.

PENDIENTE => Terminar parte desktop => Hacer parte mobile: prototipo figma + maquetación css3 + aplicativo react => generar documentación: (https://docs.google.com/document/d/1S4Iz8Dbp24YUYrukWFKRggJgspLggaDMpKqSx9n6ePw/edit)