https://i.juick.com/photos-1024/2876431.... Жуйк перестал прокручивать к новому сообщению по клику на цифру. В логе только такое: "Uncaught TypeError: t.childNodes[0].scrollIntoView is not a function"
2017-07-04 21:04:23

Участники:
@killy - 3, @karp - 2, @vt - 1, @33 - 1

@killy
Кажется, я по этим граблям уже ходил :)
#2876431/1 2017-07-04 21:23:39
@karp
Почини!
#2876431/2 → /1 2017-07-04 21:31:58
@killy
Уже закоммитил патч. Осталось подождать, пока vt его на прод выкатит.
#2876431/3 → /2 2017-07-04 21:33:34
@karp
👍
#2876431/4 → /3 2017-07-04 21:34:45
@33
Опять жаваскрипт пироги с грибами складывает. Где строгая типизация?
#2876431/5 2017-07-05 01:39:20
@vt
Да такого рода ошибки можно было бы линтером поймать, в 1с линтер такое ловит! Проблема только в том что у жс линтеры ловят разбаланс скобок и прочие дебильные советы дают :(
#2876431/6 → /5 2017-07-05 04:50:11
@killy
В данном случае путаница между нодами и элементами. IDE знает, что у элемента есть метод scrollIntoView. Но, увы, предполагает, что у произвольного типа тоже может быть такой метод (что мешает?). Потому и не ругается. Что-то такое в линтере могло бы быть, да. Перепутать .children и .childNodes - милое дело. Надо поискать в расширениях для eslint'а.
#2876431/7 → /5 2017-07-05 09:51:30