{"push_id":1297840247,"size":1,"distinct_size":1,"ref":"refs/heads/master","head":"a3f625b63351b4eba5f43ae989937e21a6df0cb6","before":"09a824753bd6a25da489a447fa4a3f23776d71b9","commits":[{"sha":"a3f625b63351b4eba5f43ae989937e21a6df0cb6","author":{"name":"Tim","email":"b503c8a0e84c436e30fba6ccbeec0ad9108c7ae0@bk.ru"},"message":"1. Про вот этот селектор замечание осталось:\nhttp://joxi.ru/RmzgOPkslay02O\nЧто селектор по id и так сильный и выбирает 1 элемент, поэтому там не надо искать внутри div.\n\n2. Код много где не выровнен:\nhttp://joxi.ru/n2Yz7N5TOExRA6\nhttp://joxi.ru/DmBEeoDiQ7b5rP\nhttp://joxi.ru/82Q73n5HgBObrd\nhttp://joxi.ru/Vm6beJ7TkMZkmZ\n\n3. При нажатии на кнопку удаления confirm появляется много раз.\nКак-то удалось такой баг воспроизвести\n\n4. Еще одна вещь, которой в условии нет, но надо сделать:\nКогда чекбоксом не выбрана ни одна строка, то кнопка \"удалить несколько\" должна быть disabled.\n\n5. В refillTable немного не так toggleClass использован.\nДолжно быть так, тогда не нужен буде if-else:\n\n$(this).toggleClass(\"filled\", (i + 1) % 2 === 0);\n\n$(this).toggleClass(\"unfilled\", (i + 1) % 2 !== 0);","distinct":true,"url":"https://api.github.com/repos/narzikulov/academit2/commits/a3f625b63351b4eba5f43ae989937e21a6df0cb6"}]}
{"actor":{"display_login":"narzikulov"}}