@kenreibman
Posted
Hi @AmritPal91 ! Great work :) Your site is nicely done using semantic elements!
Here are some suggestions that I have:
- I noticed that your navigation bar is not properly structured and styled. That's probably why you had a lot of trouble making it responsive, and you were forced to switch to the tablet layout very early. I always recommend using Flexbox to style your navigation. Looking at your site, I don't see any uses of Flexbox, and you will realize how easy it is to align content using Flexbox.
- Take a look at this video for making a fully responsive navigation menu using flexbox:
- Also take a look Kevin Powell's channel for countless flexbox tutorials like this:
If you are serious in learning web development, I recommend making this site again, or choosing a very similar challenge on here using Flexbox.
I hope this helps! I can tell you're on the right track, you just need the right tools :)
@AmritPal91
Posted
@kenreibman Thank you so much for your suggestion. I will try my best to improve as per your suggestion.