-
Notifications
You must be signed in to change notification settings - Fork 0
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
One x-editable package to rule them all #1
Comments
i'm ok with whatever we come up with here this package has been my biggest headache of all my packages, i've been suggesting that people use meteor-editable (which is not a wrapper at all, but a meteor targeted rewrite) instead as x-editable just doesnt work very well with meteor apps |
I've put out another package called x-editable-reactive-template which I believe addresses most, if not all, of the concerns with using x-editable with Meteor. It assumes you already have x-editable loaded some other way (either in a lib folder or through a package). It then makes a template in your |
Hi @dandv Sorry for my late reply and thanks for tackling this issue. We changed to using hallojs on our application and will not continue maintaining the package. I was trying to find out how to actually remove this package and found this issue: oortcloud/atmosphere#53 Any ideas how to that properly? |
Hey @banglashi, @arillo, @nate-strauser, @davidworkman9,
I've noticed a few versions of x-editable wrappers on Atmosphere:
https://atmospherejs.com/arillo/meteor-x-editable
https://atmospherejs.com/natestrauser/x-editable-bootstrap
https://atmospherejs.com/workman/meteor-editable
Based on the Atmosphere pages, Nate's seems to be the better maintained version.
We have a bigger problem, however - the original x-editable is no longer being actively maintained. Otherwise, I'd be happy to submit my Meteor integration PR to the author, and have direct support for Meteor in their repo, as we'll soon have from the Bootstrap guys.
How can we work together on this?
The text was updated successfully, but these errors were encountered: