Submitted
All feedback is welcome π
@T4R0TARO
Submitted
All feedback is welcome π
Submitted
All feedback is welcome π!
Submitted
All feedback is welcomed π I want to improve this function
outsideDivClose() : if the mobile menu is open, close mobile menu when clicked outside element
β Issue: when clicked on header section the mobile menu remains
-attempt1: addEventListener to 'container' but mobile menu will be included
-attempt2: addEventListener to document but mobile menu will be included
-attempt3: addEventListener to 'main' but top header is not included
//CODE
const main = document.querySelector("main");
// CLICK OUT OF NAV/ CLOSE NAV
const outsideDivClose = function(){
if(nav.classList.contains('active')){
nav.classList.remove('active')
}
}
main.addEventListener('click', outsideDivClose);
Submitted
Submitted
I am looking to see if my html structure follows best practice and if i used the aria-label properly. If there are other improvement I would love to hear them. Thank you in advanced for any feedback.
Submitted
All suggestions and pointers are welcome π. For this project I am wonder how I can refactor my js so that I am not repeating myself so much.
Submitted
Any feedback is welcome π¦Ύ if you have any pointers regarding how i can improve my js please let me know π.
Submitted
My HTML appears to be very repetitive specifically the rated-star containers. Any suggestion to a way to shorten the syntax to something simpler?
Submitted
Submitted
Submitted
Submitted
This is my first post, corrections and suggestions are welcome :).