Не получаестя сделать выпадающее меню, что не так?

Question

[vadfanhander]

hamburger.onclick = function() { 
  var myTopnav = document.getElementById('Topnav'); 
  var hamburger = myTopnav.querySelector('hamburger');

  myTopnav.classList.toggle('open');
}

<div class="menu">
			<ul class="flex topnav" id="myTopnav">
				<li><a href="#">Home</a></li>
				<li><a href="#">About</a></li>
				<li><a href="#">Servicing</a></li>
				<li><a href="#">Portfolio</a></li>
				<li><a href="#">Blog</a></li>
				<li><a href="#">Contact us</a></li>
				<li><a href="#"><img src="images/search.png" alt=""></a></li>
			</ul>
			
		</div>
		<div id="hamburger"><a href="#"><img src="images/menu.png" alt=""></a></div>
	</div>

@media screen and (max-width: 920px)
{
#hamburger a img{
	position: fixed;
	right: 0;
	top: 0;
	display:block;
}

.topnav{
	float:right;
	display: flex;
	flex-wrap: wrap;
	width: 15%;
	text-align: left;
	padding: 0 15px;
	background: black;
	line-height: 40px;
	position: fixed;
	right: 0;
	top: 0;
 	display: none;
}
.topnav a{
	color: #fff;
}
.topnav .open{
	display: block;
}
}

Answer 1

[Dmitriy Uvarov]

У вас id элемента myTopnav, а ищите вы Topnav

Comments

[vadfanhander]

я исправил, но проблема не решилась, консоль никаких ошибок не выдает, но меню не вылазит

[Dmitriy Uvarov]

Рабочая версия

Проблема в том, что вы определяете обработчик события еще до того, как нашли сам элемент. И второй момент, пробел в стилях не нужен для .topnav и .open

[vadfanhander]

огромное спасибо, 3 дня из-за этого пробела провозился))

Answer 2

[Илья]

С селекторами у вас беда..
https://codepen.io/itcentrpro/pen/PBLWLb