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

Error in Version 2.7.8: TypeError: this.id.toLowerCase is not a function #1855

Closed
NiklasLazinbee opened this issue Feb 25, 2016 · 9 comments
Closed

Comments

@NiklasLazinbee
Copy link

In version 2.7.8 I got an error: TypeError: this.id.toLowerCase is not a function. I'm using jQuery 2.2.1 but in 1.9.1 it's still the same.
By eliminate && this.id.toLowerCase() (line 2692) the script works
Beste Regards
Niklas

@alvarotrigo
Copy link
Owner

Can you provide a reprodutcion of the issue somewhere ?
I can't reproduce it in the demo page or in the basic jsfiddle.

@NiklasLazinbee
Copy link
Author

I found out the mistake, there ist an form on my side <input type="hidden" name="id" value="1" /> and name="id" is the probelm.

Beste Regards
Niklas

@alvarotrigo
Copy link
Owner

I found out the mistake, there ist an form on my side and name="id" is the probelm.

Beste Regards

Not really... http://jsfiddle.net/97tbk/1081/

@NiklasLazinbee
Copy link
Author

I have no Account to jsfiddle but without the form fullPage.js works. I only got the error when I set the option anchors:
anchors: ['start-info', 'about-info', 'service-info', 'actual-info', 'contact-info'],

When I change the line 2691 tom
var idAttr = $document.find('[id]').filter....
to
var idAttr = $document.find('id').filter....
the script works

@alvarotrigo
Copy link
Owner

That's not the solution for it, in fact what you are doing doesn't make sense.
My jsfiddle show's that what you said it's the problem is really not the problem.

I'll close the issue anyway as you can not provide an isolated reproduction.

@NiklasLazinbee
Copy link
Author

Okay, I reconstruct it in jsfiddle

https://jsfiddle.net/lazinbee/qL23zfbd/8/ https://jsfiddle.net/lazinbee/qL23zfbd/8/

In line 38 is the input field, if you remove it the script works

Optionally this is a solution

return $(this).attr('id') &amp;&amp; $(this).attr('id').toLowerCase() == name.toLowerCase();

Beste Regards
Niklas

Am 25.02.2016 um 13:56 schrieb Álvaro notifications@github.com:

That's not the solution for it, in fact what you are doing doesn't make sense.
My jsfiddle show's that what you said it's the problem is really not the problem.

I'll close the issue anyway as you can not provide an isolated reproduction.


Reply to this email directly or view it on GitHub #1855 (comment).

@alvarotrigo
Copy link
Owner

Thanks for it.
A better reproduction can be this one: https://jsfiddle.net/m5u4e6py/

@alvarotrigo
Copy link
Owner

Solved in the development version 2.7.9

@alvarotrigo
Copy link
Owner

Solved in 2.7.9.

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