search

nsdonato / front-invoices

🎨 Front de: Invoice App 🧾 | Simulación de trabajo en equipo con la comunidad de Twitch https://twitch.tv/vamoacodear y Discord https://discord.io/vamoacodear
https://front-invoices.vercel.app
MIT License
26 stars 5 forks source link

feature/task-7 #24

Closed Boguita closed 1 year ago

Boguita commented 1 year ago

Esta plantilla esta diseñada para organizar mejor nuestras ideas en el momento de realizar un Pull Request.

🧾 Invoices App Pull Request

Descripcion ✏️

Se agrego el componente NavBar responsive para Desktop,Tablet y Celular. Tambien se hizo un pequeño cambio del logo Vite por el de nuestra App

Cambios visuales 🎨

Desktop image Tablet image Mobile image

Como testear? 🐛

Boguita commented 1 year ago

Estaba viendo las screenshots y se ve perfecto. Lo unico que se hace medio raro es que tenga position fixed entonces el contenido de la app queda debajo de la navbar, entiendo que es para que se siga viendo por mas que haya scroll pero me parece que deberia ser estatico y que ocupe ese espacio y que el scroll sea solo dentro del contenido de la app.

Holaaa, si se entendio! No estaba seguro si dejarlo asi... Pensaba que podría ponerse un padding al body de manera responsive pero creo es mejor idea dejarlo estatico y que solo se pueda scrollear el resto de la page. No se cual de las dos opciones nos va a complicar mas pero me tiro mas a la que vos decis.