Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update CollapsableParagraph.js #489

Closed
wants to merge 1 commit into from
Closed

Conversation

cdvel
Copy link

@cdvel cdvel commented Apr 8, 2015

using classNames

@mtscout6
Copy link
Member

mtscout6 commented Apr 8, 2015

The examples are evaled at runtime which is what allows you to change them in the code tab and see live updates. This also means that we have some variables pre-defined for the examples. classSet is in fact the classnames module. See: https://github.com/react-bootstrap/react-bootstrap/blob/v0.20.2/docs/src/ReactPlayground.js

@mtscout6 mtscout6 closed this Apr 8, 2015
mtscout6 added a commit that referenced this pull request Apr 8, 2015
This has been causing a lot of confusion for folks since React has
depricated `classSet`. We felt at the time that changing the require
call to `require('classnames')` withing changing the variable name would
be enough. It hasn't due to the confusion people are seeing.

Fixes #489
@cdvel
Copy link
Author

cdvel commented Apr 9, 2015

Noted. It'd be nice that this was clearer (classSet >> classNames) as it seems a bit a misleading in the example. Thanks.

@mtscout6
Copy link
Member

mtscout6 commented Apr 9, 2015

I understand, that's why I followed up with 6dfcf36 😄

aryad14 pushed a commit to aryad14/react-bootstrap that referenced this pull request Oct 11, 2023
…strap#489)

* fix(package): update prop-types to version 15.7.0

* chore(package): update lockfile package-lock.json
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants