Prezados,
durante essa aula, uma classe e um elemento foram associados diretamente ao elemento HTML (tanto que tivemos de fazer um condicional para o clique).
Inclusive, ao clicar no buscar, o menu se fecha automaticamente. Eu não entendi ao certo porque isso aconteceu.
pode até fazer sentido, ele fechar, mas nem sempre esse vai ser o caso. o que fazer então?
Em resumo, associar os itens direto no HTML não é considerado uma má prática?
Desde já muito obrigado!
Olá, Antonio.
Desculpa, não entendi o qual é sua dúvida nesses pontos:
"pode até fazer sentido, ele fechar, mas nem sempre esse vai ser o caso. o que fazer então?"
"Em resumo, associar os itens direto no HTML não é considerado uma má prática?"
Você pode me explicar de outra forma?
Oi Marco,
sobre a primeira pergunta, desconsidere, foi só uma divagação do fato do clicar no botão buscar fechar a o menu.
O que eu quis dizer com: "Em resumo, associar os itens direto no HTML não é considerado uma má prática?"
Na aula, o professor sugere que a classe menu-ativo seja adicionada direto ao elemento
da página.Mais pra frente, também é sugerido que a adição da ação de clicar direto no elemento
. Isso faz que com que qualquer clique caia nesse método, já que todo mundo é filho da tag . Eu achei estranho adicionar essas ações direto a tag . Isso é o recomendado?att. Lucas
A ação você diz o onclick() por exemplo ? Não entendi muito bem, se der pra postar uma parte do código que você está com dúvida para podermos visualizar melhor.
Primeiro na aula https://cursos.alura.com.br/course/web-design-responsivo/task/5762 é recomendado fazer isso ao clicar nos botões:
document.querySelector('.menu-abrir').onclick = function() {
document.documentElement.classList.add('menu-ativo');
};
document.querySelector('.menu-fechar').onclick = function() {
document.documentElement.classList.remove('menu-ativo');
};depois na aula https://cursos.alura.com.br/course/web-design-responsivo/task/5765 recomenda-se isso:
document.documentElement.onclick = function(event) {
if (event.target === document.documentElement) {
document.documentElement.classList.remove('menu-ativo');
}
};eu acho estranho associar essa classe e eventos direto ao
( document.documentElement)
Também fiquei com a mesma dúvida, porque adicionar a classe na tag Html, e nao na div Container do Header, por exemplo.