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

Zepto Compatibility #398

Closed
mohitmayank opened this issue Oct 25, 2013 · 9 comments
Closed

Zepto Compatibility #398

mohitmayank opened this issue Oct 25, 2013 · 9 comments

Comments

@mohitmayank
Copy link
Contributor

Any way to make the plugin compatible with zepto. As of now it fails in adding tablesorter to zepto's namespace. line no 1277.

@Mottie
Copy link
Owner

Mottie commented Oct 25, 2013

Hi @mohitmayank!

I wouldn't mind working on support for Zepto, but with all the other work I have ahead of me, I don't think I want to try to port over the jQuery UI components used by the filter formatter; unless they have already been done?

I'll try to look at this next week. Thanks for reminding me!

@mohitmayank
Copy link
Contributor Author

initially we can ignore that part. right? i can help in porting the filter
formatter in due course.

On Sat, Oct 26, 2013 at 12:02 AM, Rob G [email protected] wrote:

Hi @mohitmayank https://github.com/mohitmayank!

I wouldn't mind working on support for Zepto, but with all the other work
I have ahead of me, I don't think I want to try to port over the jQuery UI
components used by the filter formatter; unless they have already been done?

I'll try to look at this next week. Thanks for reminding me!


Reply to this email directly or view it on GitHubhttps://github.com//issues/398#issuecomment-27115413
.

@Mottie
Copy link
Owner

Mottie commented Oct 26, 2013

Wow, this is going to be a pain...

I ended up reformating the entire plugin - not that big a deal

But then I realized I use css selectors alot! Should I force people to include the "optional" selector module, or should I try to code around that?

I haven't even touched the widgets or pager... I'm sure that'll be fun LOL.

@mohitmayank
Copy link
Contributor Author

as of now i think its a question of how much re-factoring. if its too much, using optional selector module is a good idea. if the problem is only because of jquery's selector extensions, http://api.jquery.com/category/selectors/jquery-selector-extensions/, i think a workaround will do. or you can take the parts of optional selector module which you need and include that within your code.

people use zepto cause of performance issues. for mobile apps jquery becomes a huge lib. for css selectors neither optional module nor patching approach adds too much to the user. its purely a development decision.

@mohitmayank
Copy link
Contributor Author

And i agree it will be fun. :) i started doing it myself. and ended up at the very first $.fn.extend. you re-factor the core plugin first and submit it. i will work on the widgets. as for pager, that's going to be all hell break loose.

@Mottie
Copy link
Owner

Mottie commented Oct 30, 2013

Hi @mohitmayank!

I have just pushed version 2.13.0, which includes a development branch. In that branch, I have restructured & reorganized the core plugin, then added some basic Zepto support.

Things I'm having trouble with:

  1. Getting the test-zepto.html file to pass - I just can't seem to get the initialized, sortStart, sortBegin and sortComplete events to fire on that test page. I even have the events.js file loaded.
  2. Most of the widgets should be working, except for the filter widget... I need to either figure out if zepto has nextAll and/or nextUntil or write it in plain js.

I think that's it so far.

@mohitmayank
Copy link
Contributor Author

thanks man. i just came from a long vacation. i'll see the code right away.
will contribute when i can.

@Mottie
Copy link
Owner

Mottie commented Nov 12, 2013

Welcome back! I didn't get around to updating the development branch with the changes from the last update, since I have to manually do them all now (probably doing that wrong, but I always get merge conflicts).

@mohitmayank
Copy link
Contributor Author

this is tricky. even after fighting with it for hours i have nothing to contribute. the only thing i know for sure as of now is whole zepto event isnt working in that script.

@Mottie Mottie modified the milestone: Abelt Jan 20, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants