Enhance `className` prop to accept an array

[ghsolomon]

Enhance className to accept an array that we then parse using the classnames library. This would prevent application code from needing to import / use the classnames library directly and would be an easy win.

[MalathiBalaraman31]

@ghsolomon Hi
I’d like to take this up if it’s still available.

Plan:

• update className prop type to accept string | string[]
• normalize internally using classnames
• keep backward compatibility
• add/update tests if needed

Let me know if that works .

[lbwexler]

@ghsolomon - played with this a little bit, and think it s a long time coming.

Would you incorporate the enhanced handling in to our element factory, so that all elements would benefit from the consistent syntax? Or would you only apply this to HoistComponents (i.e via HoistProps).

On the fence about this question.

[lbwexler]

@ghsolomon and I discussed -- this would be weird if we just did this on HoistComponent, leaving it not working on native HTML components.

So if you accept that conclusion, the question is: is it worth extending the small set of "features" of elementFactory to include this enhancement?