react-modal

react-modal copied to clipboard

Animation from documentation doesn't work with `shouldFocusAfterRender={false}`.

1

### Summary: Animation from documentation doesn't work on opening when `shouldFocusAfterRender={false}`. It worked when I added timeout to `ModalPortal.open`.(I do not pretend that timeout is a good solution, it is...

somename85

### Summary: I have an isOpen checking the existence of an item that the modal body requires: ```` this.setState({ item: null })} > ```` When onRequestClose is called, the modal...

akirchmyer

### Bug: Safari injects css `-webkit-overflow-scrolling: touch` and make a modal disappear. ### Steps to reproduce: 1. Open any website that use react-modal with iOS device. (not simulating on browser)...

Cerberus

Confusing props? Should we rethink them?

4

Perhaps we should consider standardizing on a set of "lifecycle" callbacks. We have some of these such as `onAfterOpen` and `onRequestClose`. I'm thinking we should deprecate/remove them in favor of...

claydiffrient

Warning about the component

11

### Steps to reproduce: Just include the lib and add it to render method... You should see: Warning: Unsafe legacy lifecycles will not be called for components using new component...

arcifius

Border appears when modal opens. Only when using className instead of inlineCss

3

### Summary: Hello, I ran into a problem using react-modal. Whenever I add a className, the weird border appears when modal open. Doesn't appear if I use inline, but I...

shmekla

[added] isNested prop

5

Changes proposed: - adds `isNested: bool` (optional) prop, which gives you the option of not removing the modal backdrop. **Why?** This enables Russian doll nesting, so you can have multiple...

Jarch09

Animation on close not properly triggered with delay

1

### Summary: ### Steps to reproduce: 1. [Open the codesandbox](https://codesandbox.io/embed/kxr4811yk3) 2. Close the modal by clicking the background 3. See no animation when closing Also when I do this in...

nealoke

[change] deprecate getParentElement in favor of local element.

9

To keep things simple, each Modal can be placed on any level of the react's tree and it will do a proper clean up when it's time to unmount the...

diasbruno

fix(#664): partial - allow empty classNames for body and portal

3

Fixes #[issue number]. Changes proposed: - - Upgrade Path (for changed or removed APIs): Acceptance Checklist: - [ ] The commit message follows the guidelines in `CONTRIBUTING.md`. - [ ]...

fortwo