@Zy8712
Posted
For a quick build this is a pretty solid site. The only things I'd change is the spacing of items from the edge of the screen (on desktop) and the choice of placing the logo outside of the main tag in your html. This site doesn't really have a footer or navbar (or much content at all) so I feel like it should just be included in the main (unless you have some specific reason to exclude it from the main).
P.S. Out of curiousity, how long did this quick build take?
@mreed4
Posted
@Zy8712 I would estimate 35-40 minutes. "A lot" of the time is spent eyeballing the sizing of everything, then getting it as close as possible. I use and pay for Copilot, so that is definitely helpful.
Thanks for the feedback on the spacing. I think the fix here is to add a padding
or margin
to body
. The generic reset I use in speed builds sometimes has unintended side effects.