Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

All comments

  • Giovanni Estrada• 250

    @GiovanniEstrada

    Posted

    Congratulations, your website looks great. actually your solution is very efficient, I just have one question: Why do you use two event listeners for the same element? I think the code can be written in the same event listener. In my case I would use the event listener like this:

    submitHandler.addEventListener("click", () => { modalOverlay.classList.toggle("hidden"); mainContainer.classList.toggle("hidden"); });

    Your code works and it's pretty good, only this part caught my attention.

    Marked as helpful

    0
  • Giovanni Estrada• 250

    @GiovanniEstrada

    Posted

    You have an amazing work, only I think the cards should be more smaller. Another tip is to try to put the overflow-x of the body in hidden to avoid problems with different resolutions. Congratulations

    Marked as helpful

    0