readme update for foundation users. #35

Merged
merged 2 commits into from Feb 12, 2014

Projects

None yet

3 participants

@iamjbecker
Contributor

I had issues getting this running until I realized the require had to go after jquery_ujs, not just jquery. Updated the docs.

Jeremy Becker added some commits Jan 19, 2014
Jeremy Becker Updated the readme.
I had issues getting this running until I realized the require had to go after jquery_ujs, not just jquery. Updated the docs.
074c17f
Jeremy Becker Foundation note.
Added a note for those who use this with Zurb Foundation.
53e8482
@iamjbecker iamjbecker closed this Feb 10, 2014
@iamjbecker iamjbecker reopened this Feb 10, 2014
@iamjbecker
Contributor

Found the real issue: the require turbolinks has to go after the foundation initialize call.

@rstacruz rstacruz merged commit dfae6f6 into kossnocorp:master Feb 12, 2014

1 check failed

default The Travis CI build failed
Details
@rstacruz
Collaborator

Thanks! merging this in and moving the node down to the Troubleshooting section.

@ryankc33

@rstacruz moving the require turbolinks after the foundation initialize call will result in turbolinks not being loaded. note

Sprockets will not look for directives in comment blocks that occur after the first line of code.

@rstacruz
Collaborator

good catch, let's change that

@rstacruz
Collaborator

I've updated the Readme to remove the Zurb section.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment