@mattstuddert
Posted
Great work again Muhammad! Your code looks great. One way you could make your Sass styles a bit easier to maintain, especially when you build larger projects, would be to nest the media queries inside the declaration blocks. So instead of this:
@media screen and (min-width: 992px) {
.menu-toggle { display: none; }
}
You could do this:
.menu-toggle {
@media screen and (min-width: 992px) {
display: none;
}
}
This keeps the styles for all screen sizes in the same area in your code, which is easier to manage. You would then use variables to define the main breakpoints.
I hope that helps! Let me know if you have any questions.
@mattstuddert thank you this is very helpful, i will apply it also to the next challange
@mattstuddert
Posted
@fadzrinmadu you're welcome. Happy to help!