Почему свойство onclick, установленное с помощью setAttribute, не работает в IE? - PullRequest
44 голосов
/ 18 сентября 2008

Сегодня столкнулся с этой проблемой, публикуя сообщения на тот случай, если у кого-то есть такая же проблема.

var execBtn = document.createElement('input');
execBtn.setAttribute("type", "button");
execBtn.setAttribute("id", "execBtn");
execBtn.setAttribute("value", "Execute");
execBtn.setAttribute("onclick", "runCommand();");

Оказывается, чтобы IE запускал клик по динамически генерируемому элементу, мы не можем использовать setAttribute. Вместо этого нам нужно установить свойство onclick для объекта с помощью анонимной функции, заключающей в себе код, который мы хотим запустить.

execBtn.onclick = function() { runCommand() };

ИДЕИ ПЛОХОГО:

Вы можете сделать

execBtn.setAttribute("onclick", function() { runCommand() });

но он не работает в IE в нестандартном режиме согласно @ scunliffe.

Вы не можете сделать это вообще

execBtn.setAttribute("onclick", runCommand() ); 

потому что он выполняется немедленно и устанавливает в качестве результата атрибута onClick результат runCommand ().

execBtn.setAttribute("onclick", runCommand);

Ответы [ 14 ]

41 голосов
/ 18 февраля 2009

, чтобы заставить это работать и в FF, и в IE, вы должны написать обоими способами:


    button_element.setAttribute('onclick','doSomething();'); // for FF
    button_element.onclick = function() {doSomething();}; // for IE

спасибо этому посту .

UPDATE : Это демонстрирует, что иногда необходимо , чтобы использовать setAttribute! Этот метод работает, если вам нужно взять исходный атрибут onclick из HTML и добавить его в событие onclick, чтобы он не переопределялся:

// get old onclick attribute
var onclick = button_element.getAttribute("onclick");  

// if onclick is not a function, it's not IE7, so use setAttribute
if(typeof(onclick) != "function") { 
    button_element.setAttribute('onclick','doSomething();' + onclick); // for FF,IE8,Chrome

// if onclick is a function, use the IE7 method and call onclick() in the anonymous function
} else {
    button_element.onclick = function() { 
        doSomething();
        onclick();
    }; // for IE7
}
10 голосов
/ 19 сентября 2008

Существует LARGE коллекция атрибутов, которую вы не можете установить в IE с использованием .setAttribute () , которая включает в себя каждый встроенный обработчик событий.

Подробнее см. Здесь:

http://webbugtrack.blogspot.com/2007/08/bug-242-setattribute-doesnt-always-work.html

7 голосов
/ 13 марта 2012

отлично работает!

использование обоих способов теперь не нужно:

execBtn.onclick = function() { runCommand() };

очевидно работает в каждом текущем браузере.

протестировано в текущих версиях Firefox, IE, Safari, Opera, Chrome для Windows; Fire Fox и Богоявление на Убунту; не тестировался на Mac или мобильных системах.

  • Крейг: Я бы попробовал "document.getElementById (ID) .type = 'password';
  • Кто-нибудь проверял подход "AddEventListener" на разных движках?
4 голосов
/ 10 июня 2010

Это удивительная функция для кросс-браузерной совместимой привязки событий.

Получил от http://js.isite.net.au/snippets/addevent

С ним вы можете просто сделать Events.addEvent(element, event, function); и не беспокоиться!

Например: (http://jsfiddle.net/Zxeka/)

function hello() {
    alert('Hello');
}

var button = document.createElement('input');
button.value = "Hello";
button.type = "button";

Events.addEvent(input_0, "click", hello);

document.body.appendChild(button);

Вот функция:

// We create a function which is called immediately,
// returning the actual function object.  This allows us to
// work in a separate scope and only return the functions
// we require.
var Events = (function() {

  // For DOM2-compliant browsers.
  function addEventW3C(el, ev, f) {
    // Since IE only supports bubbling, for
    // compatibility we can't use capturing here.
    return el.addEventListener(ev, f, false);
  }

  function removeEventW3C(el, ev, f) {
    el.removeEventListener(ev, f, false);
  }

  // The function as required by IE.
  function addEventIE(el, ev, f) {
    // This is to work around a bug in IE whereby the
    // current element doesn't get passed as context.
    // We pass it via closure instead and set it as the
    // context using call().
    // This needs to be stored for removeEvent().
    // We also store the original wrapped function as a
    // property, _w.
    ((el._evts = el._evts || [])[el._evts.length]
        = function(e) { return f.call(el, e); })._w = f;

    // We prepend "on" to the event name.
    return el.attachEvent("on" + ev,
        el._evts[el._evts.length - 1]);
  }

  function removeEventIE(el, ev, f) {
    for (var evts = el._evts || [], i = evts.length; i--; )
      if (evts[i]._w === f)
        el.detachEvent("on" + ev, evts.splice(i, 1)[0]);
  }

  // A handler to call all events we've registered
  // on an element for legacy browsers.
  function addEventLegacyHandler(e) {
    var evts = this._evts[e.type];
    for (var i = 0; i < evts.length; ++i)
      if (!evts[i].call(this, e || event))
        return false;
  }

  // For older browsers.  We basically reimplement
  // attachEvent().
  function addEventLegacy(el, ev, f) {
    if (!el._evts)
      el._evts = {};

    if (!el._evts[ev])
      el._evts[ev] = [];

    el._evts[ev].push(f);

    return true;
  }

  function removeEventLegacy(el, ev, f) {
    // Loop through the handlers for this event type
    // and remove them if they match f.
    for (var evts = el._evts[ev] || [], i = evts.length; i--; )
      if (evts[i] === f)
        evts.splice(i, 1);
  }

  // Select the appropriate functions based on what's
  // available on the window object and return them.
  return window.addEventListener
      ? {addEvent: addEventW3C, removeEvent: removeEventW3C}
      : window.attachEvent
          ? {addEvent: addEventIE, removeEvent: removeEventIE}
          : {addEvent: addEventLegacy, removeEvent: removeEventLegacy};
})();

Если вы не хотите использовать такую ​​большую функцию, она должна работать почти для всех браузеров, включая IE:

if (el.addEventListener) { 
    el.addEventListener('click', function, false); 
} else if (el.attachEvent) { 
    el.attachEvent('onclick', function); 
} 

В ответ на вопрос Крейга. Вам нужно будет создать новый элемент и скопировать атрибуты старого элемента. Эта функция должна выполнять работу: ( источник )

function changeInputType(oldObject, oType) {
  var newObject = document.createElement('input');
  newObject.type = oType;
  if(oldObject.size) newObject.size = oldObject.size;
  if(oldObject.value) newObject.value = oldObject.value;
  if(oldObject.name) newObject.name = oldObject.name;
  if(oldObject.id) newObject.id = oldObject.id;
  if(oldObject.className) newObject.className = oldObject.className;
  oldObject.parentNode.replaceChild(newObject,oldObject);
  return newObject;
}
4 голосов
/ 01 июля 2009

Или вы можете использовать jQuery и избежать всех этих проблем:

var execBtn = $("<input>", {
       type: "button",
       id: "execBtn",
       value: "Execute"
    })
    .click(runCommand);        

jQuery также позаботится обо всех кросс-браузерных проблемах.

3 голосов
/ 24 сентября 2013

На самом деле, насколько мне известно, динамически созданные встроенные обработчики событий действительно прекрасно работают в Internet Explorer 8 при создании с помощью команды x.setAttribute(); вам просто нужно правильно расположить их в своем коде JavaScript. Я наткнулся на решение вашей проблемы (и моей) здесь .

Когда я переместил все свои операторы, содержащие x.appendChild(), на их правильные позиции (т. Е. Сразу после последней команды setAttribute в их группах), я обнаружил, что КАЖДЫЙ единственный setAttribute работал в IE8, как и предполагалось, включая все формы входные атрибуты (включая атрибуты «name» и «type», а также мои обработчики событий «onclick»).

Я нашел это довольно примечательным, так как все, что я получил в IE до того, как это сделал, - это вывод мусора по экрану и одна ошибка за другой. Кроме того, я обнаружил, что каждый setAttribute по-прежнему работает в других браузерах, поэтому, если вы просто вспомните эту простую практику написания кода, вам будет полезно в большинстве случаев.

Однако это решение не будет работать, если вам придется изменять какие-либо атрибуты на лету, поскольку они не могут быть изменены в IE после добавления их HTML-элемента в DOM; в этом случае я бы предположил, что нужно будет удалить элемент из DOM, а затем воссоздать его и его атрибуты (в правильном порядке, конечно!), чтобы они работали должным образом, а не выдавали никаких ошибок.

2 голосов
/ 17 февраля 2013
function CheckBrowser(){
    if(navigator.userAgent.match(/Android/i)!=null||
        navigator.userAgent.match(/BlackBerry/i)!=null||
        navigator.userAgent.match(/iPhone|iPad|iPod/i)!=null||
        navigator.userAgent.match(/Nokia/i)!=null||
        navigator.userAgent.match(/Opera M/i)!=null||
        navigator.userAgent.match(/Chrome/i)!=null)
        {
            return 'OTHER';
    }else{
            return 'IE';
    }
}


function AddButt(i){
    var new_butt = document.createElement('input');
    new_butt.setAttribute('type','button');
    new_butt.setAttribute('value','Delete Item');
    new_butt.setAttribute('id', 'answer_del_'+i);
    if(CheckBrowser()=='IE'){
        new_butt.setAttribute("onclick", function() { DelElemAnswer(i) });
    }else{
        new_butt.setAttribute('onclick','javascript:DelElemAnswer('+i+');');
    }
}
2 голосов
/ 18 сентября 2008

Напишите функцию в строке, и интерпретатор достаточно умен, чтобы знать, что вы пишете функцию. Сделайте это так, и предполагается, что это просто строка (технически это так).

1 голос
/ 14 августа 2009

В некоторых случаях перечисленные здесь примеры не работают для меня в Internet Explorer.

Так как вы должны установить свойство таким методом (без скобок)

HtmlElement.onclick = myMethod;

это не будет работать, если вам нужно передать имя объекта или даже параметры. Для Internet Explorer вы должны создать новый объект во время выполнения:

HtmlElement.onclick = new Function('myMethod(' + someParameter + ')');

Работает также в других браузерах.

1 голос
/ 18 сентября 2008

Вы пробовали:

    execBtn.setAttribute("onclick", function() { runCommand() });
Добро пожаловать на сайт PullRequest, где вы можете задавать вопросы и получать ответы от других членов сообщества.
...