Skip to content
This repository has been archived by the owner on Feb 13, 2019. It is now read-only.

Fully Documented and Object Oriented Fart #42

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

3den
Copy link
Contributor

@3den 3den commented May 12, 2013

This refactoring of the fartscroll.js wont change any existent behavior, the smell will be the same. When this is merged the code will be much more beautiful and we will be able to fart anywhere we want too.

// Now you can just fart!
var fart = new Fart();
fart.play();

The fartscroll method got much simpler and now we should be able to make other events fart, you can fart on the command line, or on form validations... basically now you should be able to fart anyway you like and anywhere you want.

@3den
Copy link
Contributor Author

3den commented May 12, 2013

@csinchok I think you will love this ;)

@csinchok
Copy link
Contributor

I do like this a lot. I'll be merging it shortly, with a few other additions.

@monobasic
Copy link

great effort! (y)

@3den
Copy link
Contributor Author

3den commented May 18, 2013

@monobasic thanks :)

@3den
Copy link
Contributor Author

3den commented May 20, 2013

@csinchok let me know if we need some more changes on this code, I really want to get it merged to start using fart on button clicks and form validations ;)

@3den
Copy link
Contributor Author

3den commented Jun 11, 2013

@csinchok do you want to pass this project to me?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants