nextcloud-libraries / nextcloud-vue

🍱 Vue.js components for Nextcloud app development ✌ https://npmjs.org/@nextcloud/vue
https://nextcloud-vue-components.netlify.app/
Other
214 stars 85 forks source link

Suggestion community: elaborate on sparse documentation #5662

Open DaphneMuller opened 4 months ago

DaphneMuller commented 4 months ago

Feedback I received: "I am writing a new app and the vue components style guide is really helpful, but as you go down in the list the documentation gets sparser and sparser. Compare these two:

https://nextcloud-vue-components.netlify.app/#/Components/NcActions

https://nextcloud-vue-components.netlify.app/#/Components/NcVNodes"

ShGKme commented 4 months ago

NcVNodes is quite a technical component, more a rendering utility than a UI component, it has no configuration and just does what it says — renders vnodes.

NcActions is the opposite, it is a UI element with a huge number of use cases, configuration, specific usages, and accessibility concerns, requiring detailed documentation.


Missing docs:

mickenordin commented 4 months ago

It would be helpful if something like your comment about NcVNodes were visible on the documentation page (same thing for NcRelatedResourcesPanel, NcResource, NcTeamResource, etc or other things that is not meant for general consumption). Even if it is just a paragraph that says" "This is not the UI component you are looking for" so that we as consumers of the docs know that this is not something that is useful to us.

ShGKme commented 4 months ago

@mickenordin Done for NcVNodes, see: https://github.com/nextcloud-libraries/nextcloud-vue/pull/5664

mickenordin commented 4 months ago

@mickenordin Done for NcVNodes, see: #5664

Wow! Looks really good! Huge improvement!

Great job!