My first complete Landing page using HTML SASS and JavaScript

Hi Ameya!

Overall, great work and clean code, with nice and ease-to-understand classes. Also good use of SASS.

I now have a suggestion and a question 😁

**Suggestion: ** Try building the entire header+navigation with just 1 nav element. It may seem counter intuitive but it's not all that difficult to style it and seeing your code makes me think you might have considered this already.

There is a tiny bug with your navigation: open up the mobile-menu, drag your screen to the next breakpoint and the mobile AND tablet/desktop nav still show. This is easy to fix so whatever way your prefer works fine I'm sure.

I guess it's important to note that the way I code headers (with 1 set of html) is simply due to having tried it once and i just sort of stuck to it. So there's a bucket of bias with my suggestion 😎

Question: Looking through your mixins I noticed you do a breakdown-up and -down. Why is that?

I ask because when coding I start on a mobile-first design and then I simply code for tablet, and eventually, desktop. I find this to be the simplest way -> start from the simplest design iteration (always mobile, always basic single column layout) and increase the complexity as you increase and traverse through the next breakpoints with min-width.

So I'm trying to udnerstand the benefit of having max-width also, could you please explain your logic/benefits of also having breakpoint-down?

Awesome work overall 👍

Hey, really really good job on this one. I haven't seen any issue regarding the layout. Both looks good in desktop as well in mobile.

Regarding your BEM, well I think it is fine for now, but using BEM is really to make reusable and extensible components. When using BEM, make sure that the component is reusable because that is the goal of using it. It is not just using a block then just making __ elements, it is more of using a block, providing it is general. General in a way, it could be use in other sections, and not specific for a single component, because if you do that, then that is not BEM anymore.

So, how can we Identify a possible block in our BEM principle? Well, a tip is that, reviewing the layout itself. While looking at it, take note of every section, identify similarities because those similarities will be you reusable components right. Well it somewhat tedious but that will really help you to make more good components.

Overall, you did a good job^^

Hey Ameya,

This is neat! I love it when someone takes the time to also make their solution look good on medium screens even if there's no design provided.

At first, I was thinking why not name feature-icon as feature__item__icon. But looking at feature as your top-level block, I think your naming makes more sense. May I know your thought process on this?