From: Edward Betts <edward@4angle.com>
Date: Thu, 24 May 2018 11:49:12 +0000 (+0100)
Subject: Wrap the permission label with a <label> tag
X-Git-Tag: live~2984^2
X-Git-Url: https://git.openstreetmap.org/rails.git/commitdiff_plain/196ac7c0d5c1e22e62f10d2a161723ff6eef847d?ds=sidebyside

Wrap the permission label with a <label> tag

Gives the user a larger target to toggle a permission.
---

diff --git a/app/views/oauth/authorize.html.erb b/app/views/oauth/authorize.html.erb
index 3108c551a..efb7de7ec 100644
--- a/app/views/oauth/authorize.html.erb
+++ b/app/views/oauth/authorize.html.erb
@@ -12,7 +12,7 @@
   <p><%= t '.allow_to' %></p>
   <ul>
   <% @token.client_application.permissions.each do |perm| %>
-    <li><%= check_box_tag perm.to_s, "yes", @token.read_attribute(perm) %><%= t ".#{perm}" %></li>
+    <li><%= check_box_tag perm.to_s, "yes", @token.read_attribute(perm) %><%= label_tag perm.to_s, t(".#{perm}") %></li>
   <% end %>
   </ul>
   <p><%= submit_tag t(".grant_access") %></p>