From: Simon Poole <simon@poole.ch>
Date: Fri, 26 Oct 2018 18:00:00 +0000 (+0200)
Subject: Move onclick handling of tou checkbox to ujs
X-Git-Tag: live~2597^2~14
X-Git-Url: https://git.openstreetmap.org/rails.git/commitdiff_plain/d3d03a31776ed25d2fa38d069e72bd99c0914ed1

Move onclick handling of tou checkbox to ujs
---

diff --git a/app/assets/javascripts/user.js b/app/assets/javascripts/user.js
index f898455b5..40ca866c5 100644
--- a/app/assets/javascripts/user.js
+++ b/app/assets/javascripts/user.js
@@ -124,4 +124,9 @@ $(document).ready(function () {
     $("#contributorTerms").html("<img src='" + OSM.SEARCHING + "' />");
     $("#contributorTerms").load(url);
   });
+
+  $("#read_tou").on("click", function () { 
+    var agreebtn = document.getElementById('agree');
+    agreebtn.disabled=!this.checked;
+  });
 });
diff --git a/app/views/users/terms.html.erb b/app/views/users/terms.html.erb
index 18b86dd12..7a6acbf59 100644
--- a/app/views/users/terms.html.erb
+++ b/app/views/users/terms.html.erb
@@ -47,7 +47,7 @@
   <p class="deemphasize"><%= t '.tou_explain' %></p>
   <div class="form-row">
     <label for="read_tou">
-      <%= check_box_tag 'read_tou',1,false,:onclick=>"agreebtn = document.getElementById('agree');agreebtn.disabled=!this.checked;"%>
+      <%= check_box_tag 'read_tou',1,false %>
       <%= t '.read_tou_html', :tou_link => link_to(t('layouts.tou'), "https://wiki.osmfoundation.org/wiki/Terms_of_Use", :target => :new) %>
     </label>