Проблема в том, что вы связываете событие с элементом ul. когда щелчок произошел, он взял this.parentElement (это элемент, к которому был привязан клик), так что это более высокий уровень HTML, чем вам нужно. Поместите класс каретки на элемент внутри li. И следующая проблема заключается в том, что они используют элемент span, а вы используете элемент, который выполняет перенаправление по умолчанию, поэтому также необходим stopPropagation.