-
Notifications
You must be signed in to change notification settings - Fork 32
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
Documenting dependencies when run from incorrect directory structure #89
Comments
@arthurevans should I file a doc bug, or make PR? I'm not sure how the docs are built. |
@justinfagnani Either one will do. I'm having trouble following the issue description, so if you can help with a description or PR, that'd be great. It sounds like maybe the confusion is due to not using the seed-element structure or not using polyserve to preview docs locally? |
Yes, sounds like iron-component-page expects the reusable component structure. |
To clarify, I'm of the habit of writing components like this:
If I call the iron-component-page from index.html, it documents all my dependencies including Polymer etc. I've no problem with restructuring my folder structure. I'd just like to be told to do so, instead of having to find out. |
I don't think it's where the files are, but what the import URLs look like. In |
Ah right. Yes, that makes more sense. In my deviant structure, I'm importing via |
If you have a directories like:
And you run
iron-component-page
fromindex.html
, and in youryour_component.html
you reference your dependencies likebower_components/iron-image/iron-image.html
, then the resulting documentation will include all your dependencies, including polymer itself.When, however, you move your component to
your_component/bower_components/your_component
/and change the dependencies paths to../iron-image/iron-image
then it works as expected: we only document the elements inyour_component.html
.Can this please be made explicit in the documentation and the article at https://www.polymer-project.org/1.0/tools/documentation.html ? Because it's not at the moment. And it causes pain, much pain.
The text was updated successfully, but these errors were encountered: