reactjs/react-modal

Add support for React & Node 18 (as easy as adding startTransition() on each setState)

Open

#1.019 aberto em 8 de mai. de 2023

Ver no GitHub
 (19 comments) (0 reactions) (0 assignees)JavaScript (7.294 stars) (828 forks)batch import
easygood first issue

Description

Summary:

Current react modal doesn't work with latest version of React + Node v18

Steps to reproduce:

  1. Use react-modal with latest React's and Node v18

Expected behavior:

Should work normally

Example of issue:

Breaks with the following error message:

Solution:

Simply add startTransition() wrapping each of the calls for setting the state. startTransition(() => setState("something))

Guia do colaborador

Pilha de tecnologia
javascriptreact
Domain
frontendaccessibility
Tipo Issue
bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
under 1 hour
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
active
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
clear
Prerequisites
React fundamentalsstartTransition APIreact modal codebase
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
75
Direção de pesquisa
The issue requires adding startTransition() to each setState call in the react modal component. First, identify all state updates in the source code, likely in the main modal component file. Then, wrap each setState with startTransition to make it compatible with React 18. Ensure no breaking changes and test with the latest React version.