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

Canjs not working in default android browser. #681

Closed
inderps opened this issue Jan 19, 2014 · 2 comments · Fixed by #787
Closed

Canjs not working in default android browser. #681

inderps opened this issue Jan 19, 2014 · 2 comments · Fixed by #787
Milestone

Comments

@inderps
Copy link

inderps commented Jan 19, 2014

I have created a webapp using canjs 2.0.4 and jquery 2.0.3. It works fine in chrome and other browsers. But when i run it in default android browser. It fails during can.route.ready() call

And it says undefined method 'apply' for object at line 6968 i.e.
here

if (typeof method === "function") {
   return method.apply(binding, args)
} else {
   return method;
}

in can.jquery.js

please someone look into this matter

Because of this whole routing fails, hence my webapp is not working at all

I'm wrapping my app into phonegap which internally using android default browser

@ghost ghost assigned andykant Jan 23, 2014
@andykant
Copy link
Contributor

What version of Android are you testing against?
Can you provide more detail about the code around the can.route.ready() call?

The tests currently pass when I'm running it against Android 4.1 with jQuery 2.0.3.

@inderps
Copy link
Author

inderps commented Jan 24, 2014

Hey,

I tried it on android 4.0.3 (Ice Cream Sandwich) and it didnt work.
It works fine for jelly bean (4.1).

@daffl daffl added this to the 2.0.6 milestone Feb 17, 2014
ccummings pushed a commit that referenced this issue Mar 14, 2014
Changing typeof check to a check for apply (#681)
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 a pull request may close this issue.

3 participants