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

Fix missing content of expandable-data table #931

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

QuangPB
Copy link

@QuangPB QuangPB commented Oct 24, 2020

The content of expandable-data in table is not rendering
Reproduce: https://lusaxweb.github.io/vuesax/components/table.html#expandable-data
#903

@Bazsi96
Copy link

Bazsi96 commented Nov 10, 2020

Same issue.
Anyone fixed it?

@QuangPB
Copy link
Author

QuangPB commented Nov 10, 2020

Same issue.
Anyone fixed it?

You can use this repo and build: https://github.com/QuangPB/vuesax
or waiting for someone merge this pull request :(

@Bazsi96
Copy link

Bazsi96 commented Nov 10, 2020

@QuangPB I checked the Vuesax v4 but it's not compatible with Vuesax v3.

@fcsarthur
Copy link

Awesome! I just figured this out too and came here to check if anyone also did it. Someone REALLY could aprove that

@mathijsfr
Copy link

mathijsfr commented Mar 17, 2022

I installed your fork with npm install https://github.com/QuangPB/vuesax.git#master --save, yet it still doesn't render.

I added the slot like so:

.
.
.
</vs-td>
   <template slot="expand">
     test
   </template>
</vs-tr>

@QuangPB How can I use your repo and build it for my own project? Simply installing doesn't seem to work.

@QuangPB
Copy link
Author

QuangPB commented May 13, 2022

@mathijsfr : you can build project with command
vue-cli-service build --target lib src/index.js --name vuesax

or using direct branch fix from my repo
https://github.com/QuangPB/vuesax/tree/fix

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.

4 participants