Почему не работает e.preventDefault();?

Question

[Snatch08]

Всем доброго дня! Подскажите пожалуйста, почему не работает e.preventDefault(); ?

<div class="Nouxouxv"><a href="https://qna.habr.com/tags/firsttime">QNA HABR</a></div>

document.body.addEventListener('click', function(e){

	let Neoue = e.target.parentElement;

	if(Neoue.classList.contains('Nouxouxv')){

		e.preventDefault();

		window.open(e.target.href, '_blank');
	}

}, true);

Comments

[Kovalsky]

Открывается два окна?

[Snatch08]

Kovalsky, на той вкладке, на которой был тоже переходит по ссылке. И открывается в новой вкладке

[Kovalsky]

Snatch08, а если убрать window.open?

Или попробуйте например повесить обработчик не на всю страницу, а конкретно на каждую ссылку

[Snatch08]

Kovalsky, если убрать window.open, то e.preventDefault(); работает

[Snatch08]

Kovalsky, разобрался, на ссылке id висел, на котором висела функция.. благодарю, что направили к правильному решению

[historydev]

Kovalsky, дневник шкипера, ковальски снова нашёл решение за пару секунд, запись #2345 :D

[Kovalsky]

Владимир, так точно, не стоит недооценивать простую силу пингвиньей неотразимости

Snatch08, вынесите тогда в ответ

[Snatch08]

Kovalsky, я бы с радостью, но коммент же невозможно отметить решением.. напишите в ответ на вопрос, а я помечу как решение

[Kovalsky]

Snatch08, не, я имею в виду ответьте на свой вопрос сами, чтоб кто-нибудь когда-нибудь нашел и ему это было полезно

[Snatch08]

Kovalsky, аа, ок

Answer 1

[Snatch08]

с кодом все ок, проблема была с навешанной на адишник функцией, которая в этом коде не указанна

Answer 2

[Артур Ш]

Вы повесили обработчик на тэг body, почему же вы считаете что кликаете именно по ссылке, а не по соседнему или родительскому элементу? Вполне может быть, что если сделать так

let links document.querySelectorAll('.Nouxouxv a');
for(i = 0; i < links.length; i++){
   links[i].addEventListener('click', function(e){
       e.preventDefault();
       window.open(e.target.href, '_blank');
  }, true);
}

будет работать лучше.

Comments

[Snatch08]

на ссылке id висел, на котором висела функция.. благодарен за помощь

[Олег]

В скрипке вашей https://jsfiddle.net/Snatch08/fe8hv4gs/70/
js код поменяйте

document.body.addEventListener('click', function(e){ //mouseover

		if(e.target.classList.contains('Button')){

				let Awegu = 'inset 0 0 0 4px #F77816';
        
        
        document.querySelectorAll('.Btt1, .Rwx2, .Gru3, .Hrt4').forEach(function(el){
        el.style.boxShadow=Awegu;
        });
		}

}, true);

querySelectorAll - выдаёт вам массив объектом, а вы пытаетесь работать, как с уже готовым объектом
и доступ к css box-shadow должен быть таким el.style.boxShadow

[Snatch08]

Олег, аа, понял, массив. Благодарю за помощь

let Awegu = 'box-shadow: inset 0 0 0 4px #F77816';  

document.querySelectorAll('.Btt1, .Rwx2, .Gru3, .Hrt4').forEach(function(el){
  el.style = Awegu;
});

- а так же тоже можно стиль назначить, не?

[Олег]

Snatch08, стили вот так навешиваются
https://learn.javascript.ru/styles-and-classes

elem.style.backgroundColor
elem.style.width

[Snatch08]

Олег, аа, ок

[Олег]

Snatch08, классный ресурс, советую его всегда смотреть при ошибках.

[Snatch08]

Олег, да, посматриваю бывает