I'm trying to make a function that causes the buttons to only change the data-attributes of the corresponding value menu, however, it does not display an error in the console, but it does not work either:
function menu(){
var doc = document.documentElement;
var openMenus = document.querySelectorAll("button[data-menu=open]");
var closeMenus = document.querySelectorAll("button[data-menu=close]");
var menus = document.querySelectorAll("ul[data-collapse=collapse-menu]");
function openMenu(i){
if(openMenus[i].dataset.menuId == i){
menus[i].dataset.menuActive = "active";
document.documentElement.dataset.menuActive="active";
}
}
function closeMenu(i){
if(closeMenus[i].dataset.menuId == i){
menus[i].dataset.menuActive = "";
document.documentElement.dataset.menuActive="";
}
}
for(var i = 0; i < menus.length; i++){
openMenus[i].onclick = openMenu(i);
closeMenus[i].onclick = closeMenu(i);
}
}
/*Default Styles*/
*{
margin:0;
padding:0;
text-decoration:none;
color:inherit;
background-color:transparent;
box-sizing:border-box;
}
.sr-only{
position:absolute;
width:1px;
height:1px;
padding:0;
margin:-1px;
overflow:hidden;
clip:rect(0,0,0,0);
border:0;
}
.container{
display:flex;
align-items:center;
justify-content:space-between;
width:100%;
max-width:1480px;
padding:8px 20px;
margin:0 auto;
position:relative;
}
nav{
width:100%;
display:flex;
align-items:center;
}
.menu-button-holder{
display:none;
}
/*Default Media Queries*/
/*Aqui viria uma media querie com min-width 1024px, removi para que pudessem testar na resolução do computador*/
.menu-button-holder{
display:inline-block;
}
ul[data-collapse="collapse-menu"]{
position:absolute;
min-height:100vh;
top:0;
left:-100%;
padding:8px 0;
background-color:#FFFFFF;
width:90%;
z-index:10;
transition:all 1s ease-in-out;
}
html[data-menu-active="active"] ul[data-collapse="collapse-menu"][data-menu-active="active"]{
left:0;
}
html[data-menu-active="active"]:before{
content:"";
position:absolute;
top:0;
left:0;
right:0;
bottom:0;
background-color:rgba(0,0,0,.4);
z-index:5;
opacity:0;
}
<!DOCTYPE html>
<html lang="en" data-menu-active="">
<head>
</head>
<body>
<div>
<!-- Navbar -->
<nav>
<div class="container">
<div>
<img src="" alt="logo">
</div>
<div class="menu-button-holder">
<button data-menu="open" data-menu-id="0"><span class="sr-only">Abrir menu</span><svg style="width:24px;height:24px" viewBox="0 0 24 24"><path fill="#000000" d="M3,6H21V8H3V6M3,11H21V13H3V11M3,16H21V18H3V16Z" /></svg></button>
</div>
<ul data-collapse="collapse-menu" data-menu-id="0" data-menu-active="">
<div class="menu-button-holder">
<button data-menu="close" data-menu-id="0"><span class="sr-only">Fechar menu</span><svg style="width:24px;height:24px" viewBox="0 0 24 24"><path fill="#000000" d="M19,6.41L17.59,5L12,10.59L6.41,5L5,6.41L10.59,12L5,17.59L6.41,19L12,13.41L17.59,19L19,17.59L13.41,12L19,6.41Z" /></svg></button>
</div>
<li><a href="">Início</a></li>
<li><a href="">Serviços</a></li>
<li><a href="">Contato</a></li>
</ul>
</div>
</nav>
<!-- Main Content -->
<div></div>
<!-- Footer -->
<div></div>
</div>
</body>
</html>
I also tried using the setAttribute functions.
What's wrong? Why does it occur, and how can I resolve it?