Add possibility to build classname based on string about classnames HOT 9 CLOSED

We're using ES6's computed values (Babel, of course):

{
   [styles.Bar]: true,
   [this.props.className]: this.props.className,
}

from classnames.

@oleronning can you give an example of how you think this would work?

from classnames.

@dcousens based on your current implementation, this may not be straigh forward, but one possible solution is to pass an object as a parameter like so:

{
  'substrings': ['alert-', alert_type],
  'build_string': typeof(alert_type) !== 'undefined'
}

This was my first thoughts, but I'm not suprised if there is an more elegant/effective way of doing it.

Thanks!

from classnames.

@oleronning and the resultant string would be?

from classnames.

@dcousens if var alert_type = 'warning' the resultant string would be alert-warning

from classnames.

Why wouldn't you just use { "alert-warning": typeof(alert_type) !== 'undefined' }.

from classnames.

Because alert_type is passed to my method as an argument, and might just as well be danger or success

from classnames.

@ojame Sweet! Thanks a lot!

from classnames.

@ojame of course, @oleronning this is definitely the way to do this.
Lets not over-complicate this module with what is essentially a language hitch.

Thanks @ojame

from classnames.