reactjs/react-modal

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

Open

#1.019 geöffnet am 8. Mai 2023

Auf GitHub ansehen
 (19 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)JavaScript (7.294 Stars) (828 Forks)batch import
easygood first issue

Beschreibung

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))

Contributor Guide

Tech Stack
javascriptreact
Domain
frontendaccessibility
Issue Type
bug
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
under 1 hour
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
active
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
clear
Voraussetzungen
React fundamentalsstartTransition APIreact modal codebase
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
75
Research-Richtung
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.