-
Notifications
You must be signed in to change notification settings - Fork 14
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
Support for Bulma v0.6.0 #15
Comments
Seems to still not work after running Same counts for the other vue-bulma components. |
@KakaCarrotCake have you checked the status of #12 ? If you did, you'd probably realize that it still haven't been merged. If you want to fix the error, merge the changes into your node_modules by hand. Also, FWIW, other vue-bulma components have separate maintainers and trackers. You won't get any support for them here. |
@FyiurAmron Ah I see, I saw that the changes were accepted so I thought they were supposed to be pushed, I'm not too active on github issues so I didn't know; thanks for clarifying that. |
@KakaCarrotCake My issue was just about merging the PR. I am already using the fix fork. |
@MannikJ frankly, it's probably because the vue-bulma components are maintained for free by a bunch of volunteers, so any real support for them is nearly nonexistent even in the simplest cases (the so-called "fork yourself" philosophy). Not much can be done about it, apart from using those aforementioned forks (or your very own ones) instead of the root vue-bulma repo. Still, yes, if it's about #13 & #12 , then this issue is a clear duplicate and should be closed. |
Hi,
It would be really nice if someone could adjust the package to be compatible with the new version of Bulma.
The changes shouldn't be too complicated, but at the moment I'm not able to do it myself.
Ok, I just saw there is already an open pull request, that fixes this issue! Would be great if tt would be merged in the near future.
Best regards,
Jannik
The text was updated successfully, but these errors were encountered: