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

Documentation Issue #180

Open
MathisonProjects opened this issue Apr 19, 2019 · 5 comments
Open

Documentation Issue #180

MathisonProjects opened this issue Apr 19, 2019 · 5 comments

Comments

@MathisonProjects
Copy link

MathisonProjects commented Apr 19, 2019

I found your package off of NPM and spent some time working with it.

Your documentation appears to not be correct.

You have items like "Chrome" undercased which led me to being confused for a little bit.

It would be helpful to future developers if you included all of the variants of imports, and the correct way to apply them as components, eg with capitalization.

Thank you for creating such a useful addon!

@rochabianca
Copy link

I was also confused by the "Chrome" undercased, thanks for your comment, I would take a while to find out if it wasn't for it :D

@MathisonProjects
Copy link
Author

Glad to have helped!

@linx4200
Copy link
Collaborator

linx4200 commented Feb 28, 2020

Hi @MathisonProjects , sorry for the late reply. I am not a native speaker so i have a hard time understanding why you are confused by "Chrome" undercased.
Could you please explain more ? Thanks so much.

@CGraabaek
Copy link

@linx4200 the reason he's confused is:

import { swatches } from "vue-color"; as you state in the example.html will create an error, whereas import { Swatches } from "vue-color"; will be correct.

@linx4200
Copy link
Collaborator

linx4200 commented Apr 8, 2020

@CGraabaek Thanks for clarifying for me. Will correct the doc in a few days!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants