Adding the new event page to requirejs

Author: weaverryan

README.md

@@ -23,4 +23,5 @@ RequireJS
   In our case, jquery.blockUI was being brought in manually, and it was
   using define() to add itself as a module. But, I believe you can't do
   this - it's being brought in too early (or something)
-- bower install requirejs-domready --save
+- bower install requirejs-domready --save
+- organized things into pages and modules

src/Yoda/EventBundle/Resources/views/Event/new.html.twig

@@ -1,15 +1,11 @@
 {% extends 'EventBundle::layout.html.twig' %}
 
-{% block javascripts %}
+{% block requirejs %}
     {{ parent() }}
 
-    <script src="{{ asset('assets/js/NewEventApp.js') }}"></script>
-
-    <script>
-        jQuery(document).ready(function() {
-            new NewEventApp($('.event-edit'));
-        });
-    </script>
+    {{ include('EventBundle::_requirejs.html.twig', {
+        'data_main': 'app/event_new'
+    }) }}
 {% endblock %}
 
 {% block body %}

web/assets/js/app/event_new.js

@@ -0,0 +1,3 @@
+define(['jquery', 'domReady!', 'app/modules/event_new_form'], function ($, doc, EventNewForm) {
+    var eventsNew = new EventNewForm($('.event-edit'));
+});

web/assets/js/app/homepage.js

@@ -1,3 +1,3 @@
-define(['jquery', 'domReady!', 'app/modules/events_edit'], function ($, doc, EventsEdit) {
+define(['jquery', 'domReady!', 'app/modules/events_inline_edit'], function ($, doc, EventsEdit) {
     var eventsEdit = new EventsEdit($('.events'));
 });

web/assets/js/NewEventApp.js web/assets/js/app/modules/event_new_form.js

@@ -1,8 +1,4 @@
-/**
- * Application for the new event form page
- */
-(function($, window, undefined){
-
+define(['jquery', 'bootstrap'], function ($) {
     var app = function($eventsWrapper) {
         this.$wrapper = $eventsWrapper;
 
@@ -11,7 +7,6 @@
 
     $.extend(app.prototype, {
         initialize: function() {
-            console.log(this.$wrapper);
             this.$wrapper.on('submit', '.js-event-form', $.proxy(this._handleFormSubmit, this));
         },
 
@@ -50,6 +45,5 @@
         }
     });
 
-    window.NewEventApp = app;
-
-})(jQuery, window);
+    return app;
+});

web/assets/js/app/modules/events_edit.js web/assets/js/app/modules/events_inline_edit.js

@@ -10,6 +10,7 @@ requirejs.config({
     paths: {
         app: 'app',
         domReady: '../vendor/requirejs-domready/domReady',
-        jquery: '../vendor/jquery/jquery.min'
+        jquery: '../vendor/jquery/jquery.min',
+        bootstrap: '../vendor/bootstrap/dist/js/bootstrap.min'
     }
 });