From cc8d2333ed3ca1a80d1c06349abfe36e565e7b9d Mon Sep 17 00:00:00 2001
From: mo khan <mo@mokhan.ca>
Date: Fri, 17 Mar 2017 21:36:09 -0600
Subject: extract behaviour for opening a modal.

---
 app/assets/javascripts/lib/behaviours/close_modal.js.coffee |  8 ++++++++
 app/views/registrations/new.html.erb                        | 12 ++----------
 2 files changed, 10 insertions(+), 10 deletions(-)

diff --git a/app/assets/javascripts/lib/behaviours/close_modal.js.coffee b/app/assets/javascripts/lib/behaviours/close_modal.js.coffee
index f3e6ad7..28c34c7 100644
--- a/app/assets/javascripts/lib/behaviours/close_modal.js.coffee
+++ b/app/assets/javascripts/lib/behaviours/close_modal.js.coffee
@@ -4,3 +4,11 @@ class CloseModal extends Stronglifters.Behaviour
   execute: ->
     $('.close-modal').on 'click', ->
       $('.modal').removeClass('is-active')
+
+class OpenModal extends Stronglifters.Behaviour
+  @on "turbolinks:load"
+
+  execute: ->
+    $('button[data-modal]').on 'click', (event) ->
+      modalSelector = $(event.target).data('modal')
+      $(modalSelector).addClass('is-active')
diff --git a/app/views/registrations/new.html.erb b/app/views/registrations/new.html.erb
index 6a78f05..e4812af 100644
--- a/app/views/registrations/new.html.erb
+++ b/app/views/registrations/new.html.erb
@@ -22,12 +22,12 @@
           </span>
         </p>
         <%= f.label :terms_and_conditions do %>
-          <%= f.check_box :terms_and_conditions, required: 'required', "v-model": "terms_and_conditions", "v-bind:class": "{ 'is-danger': !validation.terms_and_conditions, 'is-success': validation.terms_and_conditions }" %>
+          <%= f.check_box :terms_and_conditions, required: 'required', "v-model": "terms_and_conditions" %>
           <%= t('.terms_and_conditions') %>
         <% end %>
         <p class="control">
           <%= f.submit t('.register_button'), class: "button is-success is-large is-fullwidth", ":disabled": "!isValid"  %>
-          <button id="terms-modal" type="button" class="button is-link is-pulled-left"><%= t('.terms_and_conditions_link') %></button>
+          <%= button_tag t('.terms_and_conditions_link'), type: :button, class: "button is-link is-pulled-left", data: { modal: ".modal" } %>
           <%= link_to t('.login_link'), new_session_path, class: "button is-link is-pulled-right" %>
         </p>
       <% end %>
@@ -53,11 +53,3 @@
     </div>
   </div>
 </div>
-
-<% content_for :javascript do %>
-$(function(){
-  $('#terms-modal').on('click', function(){
-    $('.modal').addClass('is-active');
-  });
-});
-<% end %>
-- 
cgit v1.2.3