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.

Sign up for GitHub

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

Jump to bottom

Documentation references bx--text__input which is incorrect #5237

Closed
josh-tt opened this issue Jan 31, 2020 · 6 comments · Fixed by #5277
Closed

Documentation references bx--text__input which is incorrect #5237

josh-tt opened this issue Jan 31, 2020 · 6 comments · Fixed by #5277
Assignees
@tw15egan

Comments

@josh-tt
Copy link

josh-tt commented Jan 31, 2020

Carbon website Form documentation references bx--text__input which is incorrect. Should be .bx--text-input
@tw15egan tw15egan transferred this issue from carbon-design-system/carbon Jan 31, 2020
@tw15egan
Copy link
Member

Nevermind, this seems to be coming from the generated docs in the main repo

@tw15egan tw15egan transferred this issue from carbon-design-system/carbon-website Jan 31, 2020
@tw15egan
Copy link
Member

Seems like a bunch of these FAQs is out of date on the website. Wondering if they should just be removed..?

Wondering what @carbon-design-system/developers think

@joshblack
Copy link
Contributor

I'd be down!

@asudoh
Copy link
Contributor

asudoh commented Jan 31, 2020

Wondering if they should just be removed..?

Yeah makes sense. Thank you for jumping in @tw15egan!

@tw15egan tw15egan self-assigned this Feb 3, 2020
@tw15egan
Copy link
Member

tw15egan commented Feb 3, 2020

Do we want to remove them from the repo completely, or do we just want to hide them on the website?

@asudoh
Copy link
Contributor

asudoh commented Feb 4, 2020

Good question, I think we can remove this, given we can bring the content back any time from the Git history.

@tw15egan tw15egan mentioned this issue Feb 5, 2020
Merged
@tw15egan tw15egan mentioned this issue Feb 28, 2020
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@tw15egan tw15egan

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

docs(readme): remove out of date readmes, migration docs tw15egan/carbon
4 participants
@asudoh @joshblack @tw15egan @josh-tt