From e834ac5051559c94cce45e4875a4a7a467941b32 Mon Sep 17 00:00:00 2001
From: Alberto Garcia Cabeza <alberto@decabeza.es>
Date: Mon, 19 Dec 2016 14:03:39 +0100
Subject: [PATCH] adds labels for attribute on advanced search form

---
 app/views/shared/_advanced_search.html.erb | 12 ++++++++----
 1 file changed, 8 insertions(+), 4 deletions(-)

diff --git a/app/views/shared/_advanced_search.html.erb b/app/views/shared/_advanced_search.html.erb
index 00db85b9c..a34de7796 100644
--- a/app/views/shared/_advanced_search.html.erb
+++ b/app/views/shared/_advanced_search.html.erb
@@ -15,14 +15,14 @@
       </div>
 
       <div class="small-12 large-3 column">
-        <label><%= t("shared.advanced_search.author_type") %></label>
+        <label for="advanced_search_official_level"><%= t("shared.advanced_search.author_type") %></label>
         <%= select_tag('advanced_search[official_level]', official_level_search_options,
                       include_blank: t("shared.advanced_search.author_type_blank")) %>
       </div>
 
       <div class="small-12 large-9">
         <div class="small-12 large-4 column">
-          <label><%= t("shared.advanced_search.date") %></label>
+          <label for="js-advanced-search-date-min"><%= t("shared.advanced_search.date") %></label>
           <%= select_tag('advanced_search[date_min]', date_range_options,
                         include_blank: t("shared.advanced_search.date_range_blank"),
                         id: 'js-advanced-search-date-min') %>
@@ -31,14 +31,18 @@
         <div id='js-custom-date' class='small-12 large-8 column' style="display: none">
           <div class="row">
             <div class='small-12 large-6 column'>
-              <label><%= t("shared.advanced_search.from") %> (<%= t("shared.advanced_search.date_placeholder") %>)</label>
+              <label for="js-advanced-search-date-min">
+                <%= t("shared.advanced_search.from") %> (<%= t("shared.advanced_search.date_placeholder") %>)
+              </label>
               <%= text_field_tag 'advanced_search[date_min]',
                                   params[:advanced_search].try(:[], :date_min),
                                   placeholder: t("shared.advanced_search.date_placeholder"),
                                   class: 'js-calendar' %>
             </div>
             <div class='small-12 large-6 column'>
-              <label><%= t("shared.advanced_search.to") %> (<%= t("shared.advanced_search.date_placeholder") %>)</label>
+              <label for="advanced_search_date_max">
+                <%= t("shared.advanced_search.to") %> (<%= t("shared.advanced_search.date_placeholder") %>)
+              </label>
               <%= text_field_tag 'advanced_search[date_max]',
                                   params[:advanced_search].try(:[], :date_max),
                                   placeholder: t("shared.advanced_search.date_placeholder"),