Я думаю, что ваш селектор должен быть:
$(".portfolio > ul > li:nth-child("+i+")").click();
... например, вы ищете nth li
, который является потомком ul
, который является потомком .portfolio
. Это дочерние селекторы. Возможно, вы могли бы использовать селектор потомков вместо этого, но я думаю (предположим), что дочерние селекторы будут немного более эффективными, поскольку у них меньше поиска. (Конечно, они также будут более хрупкими, если вы измените свою структуру.)
См. Также точку зрения Патрика о вашем i<4
сравнении, возможно, оно на единицу меньше.