KurwaTM

ul { list-style: none; }

<ul>
  <li>1</li>
  <li>2</li>
</ul>

ul li { list-style-type: none; }

Например, так (не знаю, что заставляет вас делать кнопки инпутами)
https://jsfiddle.net/pgcqjx2h/

<button type="button" data-ok="1" value="1">1</button>
<button type="button" data-ok="2" value="2">2</button>
<button type="button" data-ok="3" value="3">3</button>
<button type="button" data-ok="4" value="4">4</button>

let btns = document.querySelectorAll('[data-ok]');
btns.forEach(btn => btn.addEventListener('click', (e) =>  console.log(e.target.dataset.ok)))

li div {
    display: inline-block;
    float: right;
    margin-right: 5px;
}

Функция ID() при каждом вызове генерирует уникальное значение. В результате у нового элемента id и textContent - разные. А значит и удалить элемент по его id получится, только если подсмотреть id в HTML-коде.

Используй flex-ы или гриды.