Size: a a a

React — русскоговорящее сообщество

2020 July 05

🧨

🧨 ethorz in React — русскоговорящее сообщество
eden lane
не, тогда скроллиться не будет
почему нет?
источник

el

eden lane in React — русскоговорящее сообщество
🧨 ethorz
почему нет?
я бы попробовал воспроизвести но мне обновление макос до беты сломало codesadnbox -__-
источник

NM

Never Mind in React — русскоговорящее сообщество
Александр Бакиматов
я б на твоем месте пока вообще об оптимизациях не думал
Да я просто хоетл изначально учиться писать правильно. Вы лучше совет дайте по моему вопросу)
источник

DS

Dmitriy Shuleshov in React — русскоговорящее сообщество
Never Mind
В любом случае проблема не в этом
Обработка клика должна быть там где ты обьявлял хендлер, те выше по компонентному дереву
источник

🧨

🧨 ethorz in React — русскоговорящее сообщество
eden lane
я бы попробовал воспроизвести но мне обновление макос до беты сломало codesadnbox -__-
мб проблема из-за z-index?
источник

АБ

Александр Бакиматов... in React — русскоговорящее сообщество
Never Mind
Да я просто хоетл изначально учиться писать правильно. Вы лучше совет дайте по моему вопросу)
bind для начала убери, а потом передавай функции только те аргументы которые она ожидает
источник

el

eden lane in React — русскоговорящее сообщество
🧨 ethorz
мб проблема из-за z-index?
не, явно не в этом. помоему там есть разница в том, что именно скроллится. сам документ или блок с модалкой
источник

DS

Dmitriy Shuleshov in React — русскоговорящее сообщество
Александр Бакиматов
bind для начала убери, а потом передавай функции только те аргументы которые она ожидает
Ненужны там аргументы. Выше все должны быть сделано. А тут только ссылка на хендлер проставится должна
источник

🧨

🧨 ethorz in React — русскоговорящее сообщество
eden lane
не, явно не в этом. помоему там есть разница в том, что именно скроллится. сам документ или блок с модалкой
странно, по факту скролл никак не должен влиять, да и сам dropdown на скролл тоже
источник

АБ

Александр Бакиматов... in React — русскоговорящее сообщество
Dmitriy Shuleshov
Ненужны там аргументы. Выше все должны быть сделано. А тут только ссылка на хендлер проставится должна
это понятно, но знать о том что функции отдавать надо только то что ожидается - надо
источник

🧨

🧨 ethorz in React — русскоговорящее сообщество
Александр Бакиматов
это понятно, но знать о том что функции отдавать надо только то что ожидается - надо
поддерживаю, а то хрен знает сколько данных в props и че именно оттуда нужно
источник

И

Иван in React — русскоговорящее сообщество
Never Mind
Привет, ребят подскажите пж в чем дело. Вот в этом коде вначале срабатывает addToCart action, а потом уже выводятся пропсы из-за чего в экшн попадают неактуальные данные. Пытался через useCallBack прокинуть вторым параметром в зависимость props, но по прежнему тот же результат.
Советую почитать, что такое контекст, что такое bind и для чего он нужен. В твоём коде не нужен ВООБЩЕ.

И зачем ты снаружи передаёшь функцию, в которую нужно передать всё, что пришло снаружи? Чего бы не передать снаружи функцию, которая уже всё знает и не принимает аргументы?
источник

NM

Never Mind in React — русскоговорящее сообщество
Александр Бакиматов
это понятно, но знать о том что функции отдавать надо только то что ожидается - надо
В самом редьюсере мне нужно обновлять данные, таким способом проще делать. В прочем это не столь важно.

Я просто не могу понять почему вначале при нажатии срабатывает addToCart, а толко потом уже выводятся пропсы.
источник

АБ

Александр Бакиматов... in React — русскоговорящее сообщество
Never Mind
В самом редьюсере мне нужно обновлять данные, таким способом проще делать. В прочем это не столь важно.

Я просто не могу понять почему вначале при нажатии срабатывает addToCart, а толко потом уже выводятся пропсы.
каким таким? отдавать все пропсы которые пришли в компонент в функцию?
источник

NM

Never Mind in React — русскоговорящее сообщество
Иван
Советую почитать, что такое контекст, что такое bind и для чего он нужен. В твоём коде не нужен ВООБЩЕ.

И зачем ты снаружи передаёшь функцию, в которую нужно передать всё, что пришло снаружи? Чего бы не передать снаружи функцию, которая уже всё знает и не принимает аргументы?
Да я знаю что такое bind и зачем его использовать. Где-то слышал что через bind без передачи контекста делать более грамотно, вот и использовал
источник

АБ

Александр Бакиматов... in React — русскоговорящее сообщество
зачем тебе например в сторе ссылка на сам хэндлер клика addToCart
источник

NM

Never Mind in React — русскоговорящее сообщество
Александр Бакиматов
зачем тебе например в сторе ссылка на сам хэндлер клика addToCart
Да, возможно я и храню щас там лишние данные, но это уже совсем дургая история
источник

И

Иван in React — русскоговорящее сообщество
Never Mind
В самом редьюсере мне нужно обновлять данные, таким способом проще делать. В прочем это не столь важно.

Я просто не могу понять почему вначале при нажатии срабатывает addToCart, а толко потом уже выводятся пропсы.
Что такое «выводятся пропсы»? Куда выводятся? Кто выводит? Почему они должны сначала выводиться? А они должны выводиться?
источник

АБ

Александр Бакиматов... in React — русскоговорящее сообщество
Never Mind
Да, возможно я и храню щас там лишние данные, но это уже совсем дургая история
это все одна история - неправильный подход. который начинается с bind а заканчивается неправильными аргументами.
источник

NM

Never Mind in React — русскоговорящее сообщество
Иван
Что такое «выводятся пропсы»? Куда выводятся? Кто выводит? Почему они должны сначала выводиться? А они должны выводиться?
в консоль
источник