Почему событие не используется?

Question

У меня есть веб-страница с меню + подменю с букетом <a> анкеры. При нажатии любого из этих якорей открывается страница iframe. Он отлично работает, за исключением того, что в мобильных браузерах подменю остается открытым даже после нажатия на него, поэтому событие не «потребляется». Зачем?

Я делаю это со следующим. Во-первых, я поставил onclick функцию:

var anchors = MyMethodGetElementByClassName('loadt'); 
for(var i = 0; i < anchors.length; i++) { 
anchors[i].onclick = function(e) { 
    loadthis(e,this.href); 
} 
} 

В методе loadthis, я делаю это:

function loadthis(e,pg) 
var evt = e || window.event; 
if (evt != 88 && evt !== "null") { 
    evt.preventDefault(); 
} 
frame.src = pg; 
$('html').trigger('click'); 
return false; 
} 

Последние две строки loadthis будут стараться потреблять событие, вызывая щелчок в документ и возвращая false. Тем не менее, подменю остаются открытыми.

Посмотрите here в смартфоне (нажмите, скажем, в «Программное обеспечение> Ave»).

Answer 1

Ваш обработчик не возвращает никакой ценности. Функция, которую он вызывает (loadthis), но это не так. (Но продолжайте читать.)

Чтобы сделать обработчик возвращает значение, возвращает результат вызова loadthis:

var anchors = MyMethodGetElementByClassName('loadt'); 
for(var i = 0; i < anchors.length; i++) { 
anchors[i].onclick = function(e) { 
    return loadthis(e,this.href); 
//^^^^^^^ 
} 
} 

Кроме того, вы, как представляется, поддерживает старые браузеры. Если это так, вам нужно поставить охрану на свой вызов preventDefault (хотя на самом деле вам это не нужно, так как вы подключаете событие, return false достаточно, details in this minor post on my blog). Например .:

function loadthis(e,pg) 
var evt = e || window.event; 
if (evt != 88 && evt !== "null" && evt.preventDefault) { 
// -----------------------------^^^^^^^^^^^^^^^^^^^^^^ 
    evt.preventDefault(); 
} 
frame.src = pg; 
$('html').trigger('click'); 
return false; 
} 

Наконец: evt всегда будет != 88 и всегда будет !== "null", так что вы могли бы и, вероятно, следует удалить их.

Однако, вы, кажется, используете jQuery, из этой линии $('html').trigger('click');. Если да, то все это может быть заметно упрощен:

$(".loadt").on("click", function() { 
    $('html').trigger('click'); 
    return false; 
}); 

... так как evt != 88 && evt !== "null" всегда будет верно в любом случае ... Это будет также остановить распространение (вместо того, чтобы просто предотвратить дефолт), который может помочь.