Discussion for v4 of react-modal

First, thanks for all the support through the years…reporting and answering issues, or submitting PRs. You guys are great.

I try my best to find time to help improve this package, but I can’t do it alone…

I have a dream to finally release version 4 for react-modal, but I didn’t have a good goal to invest time to push it forward.

The goals for v4:

1. Remove from the API everything that can disable accessibility

It would be better to have a more stable releases without breaking accessibility.

1. Fix the bogus default behavior related to hide the application.

The default is to apply the aria-hidden attribute to the document.body. But this hides everything

This will require the user to explicitly define the element(s), using its id or class or element, using Modal.setAppElement().

To normalize the API, Modal.setAppElement() and appElement={} must use arrays to keep the API simple.

1. Deprecate ariaHideApp

This is an unnecessary flag, this behavior can be archived by passing [] to Modal.setAppElement() and appElement={}. But it shouldn’t be recommended.

1. Deprecate shouldFocusAfterRender={boolean}

The modal must always gain focus and trap (correctly).

1. Replace shouldReturnFocusAfterClose={boolean} with returnFocusTo={Element}

This can be an element that will gain focus after the modal is closed.

The default behavior is to return the focus to the element on document.activeElement before open the modal, but…in some browsers, this behavior is broken so we can’t always trust it. So, the user can pass to this attribute an element that must receive the focus later.

1. Review all additional systems that react-modal uses (see helpers folder)

This package manages a lot of state (like open instances, styles classes added to external elements, focus system…)

---

With this goals in mind, the API will be less flexible to disable the features related to accessibility, which should not be a problem, but maybe some users will not see this a gain…So, I decided to start a discussion before moving forward with this…

I’ll let this open until the end of Jun, and if the feedback is positive, I’ll set up the project so everybody can participate!

Thanks, Dias

[diasbruno]

@alterfo Users are free to create the animation with css. Was it difficult to add animations, or, should this package provides some options built-in?

The reason this package doesn’t provide any animations is to not add unnecessary files.

[alterfo]

add animations please, it appears roughly