Skip to content
This repository has been archived by the owner on Aug 31, 2021. It is now read-only.

Format classnames with classnames #49

Closed
prestonp opened this issue Jan 3, 2017 · 3 comments
Closed

Format classnames with classnames #49

prestonp opened this issue Jan 3, 2017 · 3 comments

Comments

@prestonp
Copy link
Contributor

prestonp commented Jan 3, 2017

@jculvey suggested using https://github.com/JedWatson/classnames to format component classnames.

I'd be happy to add it and update any modules that might need this utility

@wyuenho
Copy link
Contributor

wyuenho commented Jan 4, 2017

This could be folded into #33

@wyuenho
Copy link
Contributor

wyuenho commented Jan 11, 2017

One thing that come up in a recent internal code review is, the classnames library doesn't have good support for prefix, that alone will render 90% of the library's functionality useless. Might have to build a wrapper component to support that.

@tajo
Copy link
Contributor

tajo commented Feb 24, 2017

Closing this since we are switching to Fela.

@tajo tajo closed this as completed Feb 24, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants