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

Adding slots for empty filter results and empty dataset provided #107

Open
wants to merge 18 commits into
base: master
Choose a base branch
from

Conversation

moroq
Copy link
Contributor

@moroq moroq commented Feb 1, 2019

Tested! Hopefully it will be ok this time)

ghanbari and others added 18 commits January 8, 2019 20:57
node's text & id are flat and we can search node's id as regex too
Switching from mustache to v-html option allows to output both text and HTML message on filter "nothing found"
Add draggable class to TreeNode
Rethinking my previous PR: it is better to have wonderful component slot to display message on empty filtering result. We can get rid of emptyText prop and custom div with custom class.
And along with this: it is good to have slot for empty dataset message. Because when we fetch data from external source - we never know if there is actually any structure. So I added another slot on matches.length == 0
Adding slotted messages in TreeRoot.vue
Revert "Adding slotted messages in TreeRoot.vue"
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants