Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Open
ildar-meyker opened this issue Apr 19, 2022 · 6 comments
Labels

Comments

@ildar-meyker
Copy link

ildar-meyker commented Apr 19, 2022

<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

@JackUait
Copy link
Contributor

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

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

@ildar-meyker
Copy link
Author

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

@dzekh
Copy link
Contributor

dzekh commented May 26, 2022

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

@ildar-meyker
Copy link
Author

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

@dzekh
Copy link
Contributor

dzekh commented Jul 21, 2022

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

@zhzz
Copy link
Member

zhzz commented Jul 21, 2022

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

@dzekh dzekh changed the title RenderContainer: Warning: unstable_flushDiscreteUpdates: Cannot flush updates when React is already rendering. RenderContainer: Warning: unstable_flushDiscreteUpdates: Cannot flush updates when React is already rendering IF-458 Jul 22, 2022
@dzekh dzekh added the bug label Jul 22, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

No branches or pull requests

4 participants