tests and docs for modal helper
add some more explanation
merge conflict with readme file
added more coverage
Merge pull request #564 from toadkicker/modal_helper

Modal helper
Change JS manifest to application.js

Not sure if the naming was intentional to prevent overwriting existing files, but when the generator is run it creates a 'applications.js' file instead of an 'application.js' file. After running the generator, two JS manifest files are created which causes a asset pipeline error since one of them automatically includes the other.
Merge pull request #567 from shanepinnell/patch-1

Change JS manifest to application.js