className props should apply CSS class in any state

I know, this is the breaking change and can’t apply immediately. When this project will drastic change at v3.0 please consider following proposal.

Currently, this component can receive className and openedClassName.

• className: class name for CLOSED component
• openedClassName: class name for opened component

Generally className prop is known as that to specify class attribute always, like <div> element in the react. https://reactjs.org/docs/dom-elements.html#classname

Current behavior has given some surprising and annoying for me. IMO, following three props are LGTM.

• className: class name without condition
• closedClassName: class name for closed component
• openedClassName: class name for opened component

[karltaylor]

Fixed in v3.0.0

[iMoses-Apiiro]

@glennflanagan I’m more inline with what @karltaylor suggested, which is not dealing only with this specific request but taking the package to the next level. Basically a maintenance refactor to account for all changes in the ecosystem (hooks, modern design patterns, etc).

Although I’m personally not in favor of adding css-in-js as I think it’s too opinionated and what I love about this package is that it’s so small and doesn’t have dependencies 😃

If working hands are the issue then I can quickly wrap something up for a PR, but is it something you’d be interested in? Keeping the exact same logic but rewriting the entire API for a new major version