Удалить нажато <li> onclick

Ответ 1

ОБНОВИТЬ

Обычная делегация JS

Добавьте eventListener к UL, чтобы делегировать щелчок даже на динамически вставленных LI:

document.getElementById("ul").addEventListener("click",function(e) {
  var tgt = e.target;
  if (tgt.tagName.toUpperCase() == "LI") {
    tgt.parentNode.removeChild(tgt); // or tgt.remove();
  }
});

JQuery делегирование

$(function() {
  $("#submitButton").on("click",function() {
    var text = $("#item").val();   //getting value of text input element
    var li = $('<li/>').text(text)
    $("#ul").prepend(li); 
  });
  $("#ul").on("click","li",function() {
    $(this).remove();
  });
});   

Оригинальный ответ

Так как вы не упомянули JQuery

var listItems = document.getElementsByTagName("li"); // or document.querySelectorAll("li"); 
for (var i = 0; i < listItems.length; i++) {
  listItems[i].onclick = function() {this.parentNode.removeChild(this);}
}

Вы можете обернуть это в

window.onload=function() { // or addEventListener
  // do stuff to the DOM here
}

Перечитывая вопрос, я думаю, вы также хотите добавить это к динамическим LI.

li.innerHTML = text;    //inserting text into newly created <li> element
li.onclick = function() {
  this.parentNode.removeChild(this);
   // or this.remove(); if supported
}

Вот полный код, так как я ожидаю, что вы хотели его кодировать

Live Demo

window.onload=function() {
  var button = document.getElementById("submitButton");
  button.onclick = addItem;
}   

function addItem() {
  var textInput = document.getElementById("item");  //getting text input
  var text = textInput.value;   //getting value of text input element
  var ul = document.getElementById("ul");  //getting element <ul> to add element to
  var li = document.createElement("li");  //creating li element to add
  li.innerHTML = text;    //inserting text into newly created <li> element
  li.onclick = function() {
    this.parentNode.removeChild(this);
    // or this.remove(); if supported
  }
  if (ul.childElementCount == 0) {  //using if/else statement to add items to top of list
    ul.appendChild(li); // will add if count of ul children is 0 otherwise add before first item
  }
  else {
    ul.insertBefore(li, ul.firstChild);
  }
}

Если вы хотите использовать JQuery, все становится несколько проще

Live Demo

$(function() {
    $("#submitButton").on("click",function() {
        var text = $("#item").val();   //getting value of text input element
        var li = $('<li/>')
          .text(text)
          .on("click",function() { $(this).remove()});
        $("#ul").prepend(li); 
    });
});   

Ответ 2

Я знаю, что вы уже получили ответ, но вернитесь к своей исходной функции удаления. У вас есть следующее:

function remove(e) {
    var li = e.target;
    var listItems = document.querySelectorAll("li"); 
    var ul = document.getElementById("ul");
    li.parentNode.removeChild(li);        
}

Измените это, и вы должны получить то, что вы пытались достичь:

function remove(e)
{
   var li = e.target;
   var ol = li.parentElement;
   ol.removeChild( li);
   return false;
}

Ответ 3

Если вы не хотите писать функцию в javascript, вы можете использовать сразу вызываемую анонимную функцию, как показано ниже...

<elem onclick="(function(_this){_this.parentNode.removeChild(_this);})(this);"

Ответ 4

Я бы предложил немного упростить:

Object.prototype.remove = function(){
    this.parentNode.removeChild(this);
};

var lis = document.querySelectorAll('li');

for (var i = 0, len = lis.length; i < len; i++) {
    lis[i].addEventListener('click', remove, false);
}

JS Fiddle demo.

Конечно, выполнив вышеизложенное, я должен был пойти дальше (возможно, потому, что мне очень нравится jQuery), а также:

Object.prototype.on = function (evt, fn) {
    var self = this.length ? this : [this];
    for (var i = 0, len = self.length; i < len; i++){
        self[i].addEventListener(evt, fn, false);
    }
};
Object.prototype.remove = function(){
    var self = this.length ? this : [this];
    for (var i = 0, len = self.length; i < len; i++){
        self[i].parentNode.removeChild(self[i]);
    }
};

document.querySelectorAll('li').on('click', remove);

JS Fiddle demo.

Ответ 5

Если вы правильно поняли:

$("li").on("click", function() {
  $(this).remove()
});

Ответ 6

Ответ более очевиден, чем может показаться, вы забыли добавить init() в свой script, нормально, что событие click не запускается, они не установлены в элементе!

ИЗМЕНИТЬ

В вашем коде есть логические ошибки. Если вы не добавите функцию onclick для всех этих созданных элементов, вы не сможете удалить элемент с кликом. Это связано с тем, что функция init() вызывается один раз при загрузке страницы!

function init() {
    var button = document.getElementById("submitButton");
    button.onclick = function() {addItem()};
}

function addItem() {
    var textInput = document.getElementById("item"); //getting text input
    var text = textInput.value; //getting value of text input element
    var ul = document.getElementById("ul"); //getting element <ul> to add element to
    var li = document.createElement("li"); //creating li element to add
    li.innerHTML = text; //inserting text into newly created <li> element
    li.onclick = function() {this.parentNode.removeChild(this);}

    if (ul.childElementCount == 0) { //using if/else statement to add items to top of list
        ul.appendChild(li); // will add if count of ul children is 0 otherwise add before first item
    } else {
        ul.insertBefore(li, ul.firstChild);
    }
}
init();