@vanzasetia
Posted
👋Hi Shibu Sarkar!
First thing you need to know is that, I opened this solution on my mobile phone, which meant everything that I said was based on what I saw on my mobile device (360px width).
I have some feedback for this solution:
- The page is not 100% width, it's showing a white color on the right side.
- The header logo and the nav links are too small and the they are side by side.
- You should have email validation, you can just use CSS
:invalid
for that (no need JavaScript). - The
See how Fylo Works
link color is blue, while it should green color from thestyle-guide.md
. - The footer text is too big, try to make it smaller.
- The social icons on footer is too small.
- For the logo
alt
attribute, you cam just say the company nameFylo
, no need thelogo
. - For the useless or meaningless images, you should just leave the
alt=""
attribute empty.
That's it! Hopefully this is helpful!
Marked as helpful
@shibuwd
Posted
@vanzasetia Thank you so much for your great suggestion. I will fix it ASAP.