Vanza Setia• 27,855
@vanzasetia
Posted
👋Hi Saurabh VR!
I have some feedback on this solution:
- You should use
AOS
when you have submitted your solution. That way it will generate a better screenshot. - Don't nest too much. Only use nesting for pseudo elements (
::after
and::before
) or pseudo classes like:hover
,:active
, etc. That way you can keep the specificity low. - On my mobile device, the
main
is too narrow. - Use semantic HTML tags, like
section
for each section. - Only use one
h1
on every page. You canh2
for the other. - For the
button
inside theform
should havetype="submit"
, that way it can submit the email. Also, it's not atext
input, it's an email input. - Try to fix all issues that have been reported.
- For all
bg-section
, it's much better if you used CSSbackground
instead of an actualimg
tag.
That's it! Hopefully this is helpful!
Marked as helpful
1