diff --git a/app/views/admin/budget_investments/_advanced_filters.html.erb b/app/views/admin/budget_investments/_advanced_filters.html.erb deleted file mode 100644 index 017d8e63f..000000000 --- a/app/views/admin/budget_investments/_advanced_filters.html.erb +++ /dev/null @@ -1,45 +0,0 @@ -<%= link_to "#advanced_filters_content", - data: {toggle: "advanced_filters"}, - class: "advanced-filters float-right clear" do %> - <%= t("admin.budget_investments.index.advanced_filters") %> -<% end %> - -
- <%= form_tag(admin_budget_budget_investments_path(budget: @budget, - filter: params[:filter], - sort_by: params[:sort_by], - max_per_heading: params[:max_per_heading], - advanced_filters: params[:advanced_filters], - page: 1), method: :get, remote: true, enforce_utf8: false) do %> -
- - <%= check_box_tag "advanced_filters[]", "feasible" %> - <%= t("#{i18n_namespace}.filters.feasible") %> - - - - <%= check_box_tag "advanced_filters[]", "selected" %> - <%= t("#{i18n_namespace}.filters.selected") %> - - - - <%= check_box_tag "advanced_filters[]", "undecided" %> - <%= t("#{i18n_namespace}.filters.undecided") %> - - - - <%= check_box_tag "advanced_filters[]", "unfeasible" %> - <%= t("#{i18n_namespace}.filters.unfeasible") %> - - - - <%= t("#{i18n_namespace}.filters.max_per_heading") %> - <%= text_field_tag :max_per_heading %> - -
- -
- <%= submit_tag t("admin.budget_investments.index.buttons.filter"), class: "button expanded" %> -
- <% end %> -
diff --git a/app/views/admin/budget_investments/_search_form.html.erb b/app/views/admin/budget_investments/_search_form.html.erb index 11f99e221..c28fbb70a 100644 --- a/app/views/admin/budget_investments/_search_form.html.erb +++ b/app/views/admin/budget_investments/_search_form.html.erb @@ -1,12 +1,64 @@ -
- <%= form_for(Budget::Investment.new, url: admin_budget_budget_investments_path(budget: @budget), - method: :get, - remote: true) do |f| %> -
- <%= text_field_tag :title_or_id, "", placeholder: t("admin.budget_investments.index.placeholder") %> -
- <%= f.submit t("admin.budget_investments.index.buttons.search"), class: "button" %> +<% i18n_namespace = "admin.budget_investments.index" %> + +<%= form_tag(admin_budget_budget_investments_path(budget: @budget), method: :get, enforce_utf8: false) do %> +
+
+ <%= link_to "#advanced_filters_content", + data: {toggle: "advanced_filters"}, + class: "advanced-filters float-right clear" do %> + <%= t("#{i18n_namespace}.advanced_filters") %> + <% end %> +
+
+
+ <% ["feasible", "selected", "undecided", "unfeasible"].each do |option| %> +
+
+ <%= check_box_tag "advanced_filters[]", option, false, id: "advanced_filters_#{option}" %> + <%= t("#{i18n_namespace}.filters.#{option}") %> +
+
+ <% end %> +
+
+ <%= text_field_tag :max_per_heading, "", placeholder: t("#{i18n_namespace}.filters.max_per_heading") %>
- <% end %> -
+
+
+
+ <%= select_tag :administrator_id, + options_for_select(admin_select_options, params[:administrator_id]), + { prompt: t("#{i18n_namespace}.administrator_filter_all"), + label: false} %> +
+
+ <%= select_tag :valuator_or_group_id, + options_for_select(valuator_or_group_select_options, params[:valuator_or_group_id]), + { prompt: t("#{i18n_namespace}.valuator_filter_all"), + label: false} %> +
+
+ <%= select_tag :heading_id, + options_for_select(budget_heading_select_options(@budget), params[:heading_id]), + { prompt: t("#{i18n_namespace}.heading_filter_all"), + label: false} %> +
+
+ <%= select_tag :tag_name, + options_for_select(investment_tags_select_options(@budget), params[:tag_name]), + { prompt: t("#{i18n_namespace}.tags_filter_all"), + label: false} %> +
+
+
+
+
+ <%= text_field_tag :title_or_id, "", placeholder: t("#{i18n_namespace}.placeholder") %> +
+
+
+ <%= submit_tag t("#{i18n_namespace}.buttons.filter"), class: "button expanded" %> +
+
+<% end %> diff --git a/app/views/admin/budget_investments/index.html.erb b/app/views/admin/budget_investments/index.html.erb index f2672f564..09b6e875b 100644 --- a/app/views/admin/budget_investments/index.html.erb +++ b/app/views/admin/budget_investments/index.html.erb @@ -8,42 +8,6 @@ <%= render "search_form" %> -<%= form_tag(admin_budget_budget_investments_path(budget: @budget), method: :get, enforce_utf8: false) do %> -
- <%= select_tag :administrator_id, - options_for_select(admin_select_options, params[:administrator_id]), - { prompt: t("admin.budget_investments.index.administrator_filter_all"), - label: false, - class: "js-submit-on-change" } %> -
- -
- <%= select_tag :valuator_or_group_id, - options_for_select(valuator_or_group_select_options, params[:valuator_or_group_id]), - { prompt: t("admin.budget_investments.index.valuator_filter_all"), - label: false, - class: "js-submit-on-change" } %> -
- -
- <%= select_tag :heading_id, - options_for_select(budget_heading_select_options(@budget), params[:heading_id]), - { prompt: t("admin.budget_investments.index.heading_filter_all"), - label: false, - class: "js-submit-on-change" } %> -
- -
- <%= select_tag :tag_name, - options_for_select(investment_tags_select_options(@budget), params[:tag_name]), - { prompt: t("admin.budget_investments.index.tags_filter_all"), - label: false, - class: "js-submit-on-change" } %> -
-<% end %> - -<%= render "advanced_filters", i18n_namespace: "admin.budget_investments.index" %> - <%= render "/shared/filter_subnav", i18n_namespace: "admin.budget_investments.index" %>