Skip to content

Preserve "Cancel" button state in the swap list#486

Merged
sindresorhus merged 1 commit into
masterfrom
preserve-cancel-button-state
Aug 27, 2018
Merged

Preserve "Cancel" button state in the swap list#486
sindresorhus merged 1 commit into
masterfrom
preserve-cancel-button-state

Conversation

@sindresorhus
Copy link
Copy Markdown
Contributor

We cannot keep local state as react-virtualized unmounts components outside the viewport:

You might be aware of this, but just to be completely clear- components will not retain state once they've left the viewport. This is a limitation of windowing with react-virtualized (or similar libraries). Any "state" that should persist should be stored externally (eg within the underlying List/Array, in Redux, in the parent component's state, etc). - bvaughn/react-virtualized#416 (comment)

@sindresorhus sindresorhus changed the base branch from gtc to master August 27, 2018 09:25
We cannot keep local state as `react-virtualized` unmounts components outside the viewport:

> You might be aware of this, but just to be completely clear- components will not retain state once they've left the viewport. This is a limitation of windowing with react-virtualized (or similar libraries). Any "state" that should persist should be stored externally (eg within the underlying List/Array, in Redux, in the parent component's state, etc). - bvaughn/react-virtualized#416 (comment)
@sindresorhus sindresorhus force-pushed the preserve-cancel-button-state branch from 4c82a20 to 8bf58dc Compare August 27, 2018 09:29
@sindresorhus sindresorhus merged commit 2718a53 into master Aug 27, 2018
@sindresorhus sindresorhus deleted the preserve-cancel-button-state branch August 27, 2018 09:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant