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

Not working #1

Closed
stell opened this issue Mar 31, 2011 · 10 comments
Closed

Not working #1

stell opened this issue Mar 31, 2011 · 10 comments

Comments

@stell
Copy link

stell commented Mar 31, 2011

If i start index.html in FF4 i get:

missing ; before statement
[Break On This Error] var noobSlide=new Class({initialize:fu...ndles[this.currentIndex]:null));}}}});
_class...cked.js (line 1)

noobSlide is not defined
[Break On This Error] var nS1 = new noobSlide({
index.html (line 15)
handles8_more is undefined

@celticwolf
Copy link
Owner

I don't get this error in FF 3.6.16. What platform are you on (Windows, Linux, OS X) and what version, please?

@stell
Copy link
Author

stell commented Mar 31, 2011

It's Win XP and FF 4. Same Error in latest Chrome and IE 8.
I just unpacked the whole thing and startet index.html.

@celticwolf
Copy link
Owner

I can replicate it in IE 8 on WinXP. I'll take a look later when I have a bit of time.

@stell
Copy link
Author

stell commented Mar 31, 2011

Just wanted to state that its great you put noobSlide with fading on github. I'm working often with noobslide and if fading works it will be nice.

@celticwolf
Copy link
Owner

You're welcome. Until I have time to look at the problem, try using the nonpacked version instead: _class.noobSlide.js. It's possible that the packer I used messed something up.

@stell
Copy link
Author

stell commented Mar 31, 2011

That helped. Seams to be the packer.
I'm getting some "this.fx is undefined" issues but working on that...

@stell
Copy link
Author

stell commented Mar 31, 2011

while i'm at it. looks like play/pause ain't working in fade mode.

@celticwolf
Copy link
Owner

I tested with the nonpacked version and did not get the error. I think that the packer I used introduced that error.

@stell
Copy link
Author

stell commented Mar 31, 2011

Fading did'nt work cause addButtons Method has been renamed. Now i know ;)

@celticwolf
Copy link
Owner

I have replaced the packed version with a new version compressed using Douglas Crockford's JSMin. This version does not cause any errors in IE 8 or Chrome 10.

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

No branches or pull requests

2 participants