classNames

[jamiebuilds]

I see that react-modal used to have the ability to specify classes for styles but that was removed in favor of inline styles.

While some might argue that inline styles are better for react applications, not everyone has that option (including myself).

It would be nice if react-modal wasn't opinionated about this and supported both giving classNames and inline styles.

[paazmaya]

👍

[nbwar]

+1 please bring back classNames. Its a pain to do custom animations with inline-styles that can't be overwritten with css.

[idmontie]

👍

[jamiebuilds]

For those that are interested I've started a separate project react-modal2 which is less opinionated, accessible, stateless (dumb component), and is built via a reusable collection of modules. It also allows you to use classNames instead of inline styles.

[kjs3]

👍

[OKNoah]

Seems like very little effort to support both. 👍

[antoinerey]

👍

[simpixelated]

Looks like #100 would fix this, but it hasn't been reviewed/merged yet.

[claydiffrient]

v1.0.0 is now released with supported added back for classNames