Size: a a a

2020 September 13

AP

Alexander Ponomarev in Svelte [svelt]
Pavel 🦇 Malyshev
как видишь никакого свелт, просто 2 хендлера на один элемент.
я думал что он сделает один addEventListener и раскрутит в нем массив колбэков
источник

PM

Pavel 🦇 Malyshev in Svelte [svelt]
Alexander Ponomarev
я думал что он сделает один addEventListener и раскрутит в нем массив колбэков
Собственно на это я уже также ответил выше. Когда крутили фичу с множественными хендлерами никакой батчинг не делали и нативное поведение не меняли. Хорошо или нет, затрудняюсь ответить. С одной стороны твоя позиция ясна. С другой,  открываются новые возможности - можно получить 2 варианта поведения: в одном хендлере вызвать несколько и работать на одном рендере или сделать 2 хендлера и 2 рендера. Наверное есть кейсы в которых это даже пригодиться.
источник

AP

Alexander Ponomarev in Svelte [svelt]
хз я теперь и не знаю, такое решение имеет право на жизнь, но перф можно убить без батча на какомнить скроле.

Ну и не всегда собственно колбэки вызываются асинхронно=) https://jsfiddle.net/2fowk4r7/2/

у меня реакт деформация, спишем на это
источник

MZ

Misha Zamkevich in Svelte [svelt]
Pavel 🦇 Malyshev
собственно все предсказуемо же. надо такие вещи делать afterUpdate. А вообще по-хорошему через #if
источник

КС

Кирилл Спасибович... in Svelte [svelt]
источник
2020 September 14

MZ

Misha Zamkevich in Svelte [svelt]
Кирилл Спасибович
Потерпи, или затагай автора.
источник

MZ

Misha Zamkevich in Svelte [svelt]
Кирилл Спасибович
А вообще у тебя там такое описание конечно. Я бы написал почему именно так сделал.  Почему weird way а не нормальный. Почему нормально не получается и почему это работает и что это на самом деле норм а не какой-то говнокод.

А то из описания выходит как будто это говнокод.
источник

КС

Кирилл Спасибович... in Svelte [svelt]
Misha Zamkevich
А вообще у тебя там такое описание конечно. Я бы написал почему именно так сделал.  Почему weird way а не нормальный. Почему нормально не получается и почему это работает и что это на самом деле норм а не какой-то говнокод.

А то из описания выходит как будто это говнокод.
типа они читают описания и в код из 13 строк даже не глядят ?
источник

MZ

Misha Zamkevich in Svelte [svelt]
Даже для 1 строчки нужно обоснование. Никто не знает, что у тебя в голове.
источник

КС

Кирилл Спасибович... in Svelte [svelt]
Misha Zamkevich
Даже для 1 строчки нужно обоснование. Никто не знает, что у тебя в голове.
https://github.com/sysgears/webpack-virtual-modules/pull/66
а это подойдёт ? или там тоже недостаточно ?
источник

КС

Кирилл Спасибович... in Svelte [svelt]
я просто не могу с уверенностью говорить что это не говно, потому что то в чём я не уверен я оцениваю как говно
источник

КС

Кирилл Спасибович... in Svelte [svelt]
это фиксит проблему минимальным количеством кода, но правильно ли это?
источник

КС

Кирилл Спасибович... in Svelte [svelt]
ровно как и правильно ли делать функции плагина внутри лоадера для вебпака ?
источник

КС

Кирилл Спасибович... in Svelte [svelt]
и где Рич пишет про hacky way и что он не может avoid hacks
https://github.com/sveltejs/svelte-loader/pull/59#issuecomment-385381444
источник

MZ

Misha Zamkevich in Svelte [svelt]
Кирилл Спасибович
я просто не могу с уверенностью говорить что это не говно, потому что то в чём я не уверен я оцениваю как говно
Ну так в этом и полезное действие. Когда ты пишешь обоснование, то тебе приходиться глубже погружаться в вопрос и там могут отпасть часть вопросов. Или вообще окажется что ПР не нужен. Или что он работает только для wp4 😉
источник

КС

Кирилл Спасибович... in Svelte [svelt]
ну да, пусть пересобирается весь компонентный css на любом изменении, ПР не нужен :)
Константин прямо кайфовал от 40 секунд загрузки postcss'а
источник

MZ

Misha Zamkevich in Svelte [svelt]
Я имел ввиду не конкретно этот же)
источник

КС

Кирилл Спасибович... in Svelte [svelt]
но продолжил для wp4 )
источник

MZ

Misha Zamkevich in Svelte [svelt]
Ой, ну чё ты привязался ))
источник

КС

Кирилл Спасибович... in Svelte [svelt]
а читал ишак? или в него тож не кликнут ?
источник