Skip to content
This repository has been archived by the owner on Mar 8, 2019. It is now read-only.

Have more than one mixin per js file #87

Closed
elevatebart opened this issue Jan 27, 2019 · 1 comment · Fixed by #88
Closed

Have more than one mixin per js file #87

elevatebart opened this issue Jan 27, 2019 · 1 comment · Fixed by #88

Comments

@elevatebart
Copy link
Member

At the moment if a mixins.js file contains more than one file the document parser loads all mixins for each component.
It would be better to only load the mixin that are mentionned in the mixin property of the documented component.

example

import { mixin1, mixins2 } from './mixins'

export default {
  name: 'testMixins'
  mixins: [mixin1]
}

Here we should obtain documentation for testMixins extended with mixin1.
Instead, we get mixin1 and mixin2.

elevatebart added a commit that referenced this issue Jan 28, 2019
* make structure for exports

* make sure all exported members are named

* ready for muti mixins experimentation

* add multi mixins tests

* fix case of unit test

fixes #87
@vue-styleguidist-bot
Copy link
Collaborator

🎉 This issue has been resolved in version 3.3.4 🎉

The release is available on:

Your semantic-release bot 📦🚀

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

Successfully merging a pull request may close this issue.

2 participants