retail-ui icon indicating copy to clipboard operation
retail-ui copied to clipboard

RenderContainer: Warning: unstable_flushDiscreteUpdates: Cannot flush updates when React is already rendering IF-458

Open ildar-meyker opened this issue 2 years ago • 6 comments

<Kebab size="large"> <MenuItem icon={<EyeOpened />} onClick={() => console.log(1)} > Все </MenuItem> </Kebab>

При клике на MenuItem бросает ошибку Warning: unstable_flushDiscreteUpdates: Cannot flush updates when React is already rendering. Версия 4.0.4. и 4.0.5

ildar-meyker avatar Apr 19 '22 23:04 ildar-meyker

Привет! Спасибо за ишью. Похоже, что проблема очень старая. Получилось отследить её до 3.0.0, но очевидно, что она появилась раньше. Сейчас у нас есть две задачи, одна из которых потенциально может закрыть проблему, это задача про вложенность MenuItem из #2749, а также вторая задача, которая наверняка закроет эту проблему, это задача про перевод (и рефактор) всей библиотеки на функциональные компоненты. Первая задача будет закрыта в ближайшее время, про вторую тоже самое сказать не могу.

Подскажи, насколько критичен для вас этот баг? Если баг критичен для тебя или твоей команды, возьмём этот баг как отдельную задачу.

JackUait avatar Apr 20 '22 02:04 JackUait

В целом как бы важная штука. Можно, конечно, своим компонентом заменить, если скажете, как можно закрыть Kebab ручками. Есть ли публичные методы как у Dropdown open/close?

ildar-meyker avatar Apr 21 '22 14:04 ildar-meyker

@ildar-meyker а почему важная штука? Падает ли приложение от этой ошибки?

dzekh avatar May 26 '22 06:05 dzekh

@dzekh нет, не падает. Природа ошибки мне не ясна, не исследовал, и возможные последствия тоже. Компонент активно используется, поэтому важность обозначена лишь на уровне предположения.

ildar-meyker avatar May 26 '22 12:05 ildar-meyker

@JackUait @zhzz А эта задачка не связана с #2941 ? может она там исправлена?

dzekh avatar Jul 21 '22 12:07 dzekh

Нет, она еще присутствует.

zhzz avatar Jul 21 '22 12:07 zhzz