Responsive product landing page3

Hi guys!
I hope your are going well. and please review my landing page, I want to know my improvement, so if it pleases you give me some feedback and I will appreciate it, thanks.
Here is the link,

Hi @Diu!

I think your page looks good.

I would move the javascript out of the html and into the javascript section.

This is optional but I might personally write it like this

let menubtn = document.getElementById("menubtn");
let navbar2 = document.getElementById("navbar2");
let menu = document.getElementById("menu");
menubtn.onclick = () =>"-250px" ?"0":"-250px";

Remember to keep the test suite in the page
<script src=""></script>

You might consider adding a smooth scroll to the page

html {
  scroll-behavior: smooth;

Your page looks good @Diu. Something to revisit;

  • When the user clicks to go to the “work” section the navbar seems to disappear. White text on a white background is not a good UX.
    Clicking “deals” gives the same bad UX though a little portion of the box around login does show.
  • As mentioned, keep the test script when forking the pen (<script src=""></script>)
    • The test script should be included, with all tests passing, when you submit your projects.

Thanks so much for your kind feedback, I have changed the nav text to black so that it can appear clear in the work section, thanks once again.

Thanks my dear for your kind feedback, I have added the smooth scroll and have exported the script to it’s Js file, thanks once again.