Nishant23 / html5slides

Automatically exported from code.google.com/p/html5slides
0 stars 0 forks source link

slide.js broken for 'build' class (even in google demo) #47

Open GoogleCodeExporter opened 8 years ago

GoogleCodeExporter commented 8 years ago
What steps will reproduce the problem?
1. Attach 'build' class to any object
2. Reload project and try to hit next slide while on the slide with 'build' 
class
3. Nothing happens, you get "Uncaught Error: SYNTAX_ERR: DOM Exception 12" on 
line 220 in slide.js in console.

What is the expected output? What do you see instead?
The object should show up instead of throwing error.

What version of the product are you using? On what operating system?
Latest version. Windows 7

Please provide any additional information below.

Original issue reported on code.google.com by ty...@appmobi.com on 16 Jan 2013 at 3:23

GoogleCodeExporter commented 8 years ago
I am using Chrome Version 24.0.1312.52 m

The google demo works in Firefox but not Chrome for me. 

Original comment by ty...@appmobi.com on 16 Jan 2013 at 3:40

GoogleCodeExporter commented 8 years ago
I have the same error. 
I am using Google Chrome Versión 24.0.1312.52 m and Windows 7

Original comment by gustavo....@gmail.com on 17 Jan 2013 at 9:52

GoogleCodeExporter commented 8 years ago
slides.js

220c220
<   toBuild[0].classList.remove('to-build', '');
---
>   toBuild[0].classList.remove('to-build');

Original comment by yungs...@gmail.com on 18 Jan 2013 at 6:44

GoogleCodeExporter commented 8 years ago
Yeah this a pretty big issue that affects any slide deck using the .build class 
- please fix soon

Original comment by wes...@gmail.com on 22 Jan 2013 at 5:49

GoogleCodeExporter commented 8 years ago
Right now I have a working copy by saving slides.js onto my computer and 
implementing the fix on line 220 by changing: 
toBuild[0].classList.remove('to-build', ''); to 
toBuild[0].classList.remove('to-build');

If you need your presentation to work before they fix it, I'd suggest doing 
that, saved my presentation thanks to yungs...@gmail.com. 

Original comment by ty...@appmobi.com on 22 Jan 2013 at 6:23