@Esesosa-max
Posted
- Good Looking Design (desktop 3/3,mobile 2/3) 5/6
- HTML is somewhat structured 2.5/5
- deadly responsive (impressive !!) 5/5
- Good Application of CSS Variables and mobile first (impressive !!) 5/5
- Fixes and personal suggestions
- i do not think you should put everything into the main
- instead of using
<header>
use<nav>
or just put the nav inside the header <header><nav></nav></header>
- your header is a bit mess up
- input is not properly styled
- Great Job : 18/21
Marked as helpful
@CrypticMango
Posted
@Esesosa-max Thanks for the feedback! So I have a question... about not having everything inside of the main, it's okay to just have some elements hanging around on their own without being inside of the main? or do I have to use a different element for the rest? Not sure why but I was under the impression that I had to put everything in the main
Also, what did I do wrong with my input?
Thanks for the tips and compliments I appreciate this a lot. I have to go back and sift through some things.
@Esesosa-max
Posted
@CrypticMango very sorry for the late reply, my computer has been very terrible lately so I could not log in to my frontendmentor. The alignment of the text inside of the input; that's not styled properly. The problem with the main, this is my personal opinion but I think you should put most of the elements inside the main and the remaining elements put them inside a section and footer.