Hello!
Nice job, the desktop version looks good.
For the media query, I think you should be consistent whether you want to use min-width or max-width.
If you build your website starting from desktop layout --> mobile layout, use max-width. the larger screen on top of small screen. For example, @media(max-width:1000px){ } then, @media(max-width: 700px).
else, if you are doing mobile-first (which is the recommended way), use min-width.
for min-width, small screen followed by larger screen. For example, @media(min-width: 700px) then, @media(min-width:1000px){ } .. hope this helps!
Hye NoorAlhijab,
your website is working great!
this is only a suggestion, maybe you can make the menu in mobile view position fixed. So that if user scroll the page, they can still see the menu
Other than that, everything is amazing.
Latest Comments
RE: BENOIT Matthias's "Use html, sass and npm for compile css, js for menu mobile" solution
Hye Matthias Benoit, I cannot see your code when i clicked on “view code” button 🤔
RE: Cody's "Fylo Data Storage Component Challenge" solution
Hello! Nice job, the desktop version looks good. For the media query, I think you should be consistent whether you want to use min-width or max-width.
If you build your website starting from desktop layout --> mobile layout, use max-width. the larger screen on top of small screen. For example, @media(max-width:1000px){ } then, @media(max-width: 700px).
else, if you are doing mobile-first (which is the recommended way), use min-width. for min-width, small screen followed by larger screen. For example, @media(min-width: 700px) then, @media(min-width:1000px){ } .. hope this helps!
RE: NoorAlhijab's "Project tracking intro component" solution
Hye NoorAlhijab, your website is working great! this is only a suggestion, maybe you can make the menu in mobile view position fixed. So that if user scroll the page, they can still see the menu Other than that, everything is amazing.
RE: Fantee-hub's "created with bootstrap,css,javascript" solution
the animation is awesome!