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

Hi! I cleaned up your code for you! #11

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
8 changes: 4 additions & 4 deletions Demos/index.html
@@ -1,4 +1,4 @@
<!doctype html> <!doctype html>


<html lang="en" class="not-ready"> <html lang="en" class="not-ready">


Expand Down Expand Up @@ -31,8 +31,8 @@ <h1>MooTools SlideShow</h1>
<h2>v2.0.0</h2> <h2>v2.0.0</h2>
<p>A highly extensible interface widget</p> <p>A highly extensible interface widget</p>
<p>[ <p>[
<a href="http://github.com/rpflorence/SlideShow">Github</a> | <a href="http://github.com/rpflorence/SlideShow">Github</a> |
<a href="http://mootools.net/forge/p/slideshow">The Forge</a> | <a href="http://mootools.net/forge/p/slideshow">The Forge</a> |
<a href="http://ryanflorence.com">ryanflorence.com</a> <a href="http://ryanflorence.com">ryanflorence.com</a>
] ]
</p> </p>
Expand Down Expand Up @@ -354,6 +354,6 @@ <h2>News Ticker</h2>
s.parentNode.insertBefore(g, s); s.parentNode.insertBefore(g, s);
})(document, 'script'); })(document, 'script');
</script> </script>

</body> </body>
</html> </html>
4 changes: 2 additions & 2 deletions Demos/js/flickr.js
Expand Up @@ -12,7 +12,7 @@ Array.implement({


// Define a new transition // Define a new transition
SlideShow.defineTransition('squares', function(data){ SlideShow.defineTransition('squares', function(data){
// data contains information about the transition like the // data contains information about the transition like the
// previous and next slides as well as the duration of the transition // previous and next slides as well as the duration of the transition
var elements = data.next.getElements('span'), var elements = data.next.getElements('span'),
duration = data.duration / elements.length - 1; duration = data.duration / elements.length - 1;
Expand Down Expand Up @@ -104,7 +104,7 @@ SlideShow.Flickr = new Class({
this.initialized = true; this.initialized = true;
this.element.fade('in'); this.element.fade('in');
}, },

show: function(index, options){ show: function(index, options){
if (!this.initialized) return this; if (!this.initialized) return this;
return this.parent(index, options); return this.parent(index, options);
Expand Down
2 changes: 1 addition & 1 deletion Demos/js/navigation.js
Expand Up @@ -36,6 +36,6 @@ document.addEvent('domready', function(){
y: 20 y: 20
} }
}); });



}); });
4 changes: 2 additions & 2 deletions Demos/js/site.js
Expand Up @@ -49,8 +49,8 @@ var watcher = {
document.addEvent('domready', function(){ document.addEvent('domready', function(){
$$('html')[0].removeClass('not-ready').removeClass('no-js'); $$('html')[0].removeClass('not-ready').removeClass('no-js');
watcher.init([ watcher.init([
flickr, basic.pause(), flickr, basic.pause(),
css.pause(), css.pause(),
declarative.pause() declarative.pause()
]); ]);
}); });
2 changes: 1 addition & 1 deletion Tests/SlideShow/SlideShow.html
Expand Up @@ -62,7 +62,7 @@
<div data-slideshow="transition:fade" class=green>18<br> fade</div> <div data-slideshow="transition:fade" class=green>18<br> fade</div>
<div data-slideshow="transition:fadeThroughBackground" class=red>19<br> fadeThroughBackground</div> <div data-slideshow="transition:fadeThroughBackground" class=red>19<br> fadeThroughBackground</div>
<div data-slideshow="transition:flash" class=blue>20<br> custom</div> <div data-slideshow="transition:flash" class=blue>20<br> custom</div>

<p>I'm not really a part of this</p> <p>I'm not really a part of this</p>
</div> </div>


Expand Down