diff --git a/app/views/reimbursement/reports/_edit_form.html.erb b/app/views/reimbursement/reports/_edit_form.html.erb index 04d7e5edc1..f02128a89c 100644 --- a/app/views/reimbursement/reports/_edit_form.html.erb +++ b/app/views/reimbursement/reports/_edit_form.html.erb @@ -1,45 +1,66 @@ <%= form_with(model: @report, local: true) do |form| %> - <%= form.label :report_name, "Report name" %> - <%= form.text_field :report_name, placeholder: "Chicago Trip Expenses", required: true %> - <%= form.label :report_to, "Report to", class: "mt2" %> - <%= form.text_field :report_to, value: @report.user.email, readonly: true %> +
+
+ <%= form.label :report_name, "Report name" %> + <%= form.text_field :report_name, placeholder: "Chicago Trip Expenses", required: true %> +
+
+ <%= form.label :report_to, "To" %> + <%= form.text_field :report_to, value: @report.user.email, readonly: true %> +
+
+ <% if organizer_signed_in? && (@report.submitted? || @report.draft?) %> - <%= form.label :maximum_amount, "Maximum reimbursable value (optional)", class: "mt2" %> + <%= form.label :maximum_amount, "Maximum reimbursable value", class: "mt2" %>
-
+
$ - <%= form.number_field :maximum_amount, placeholder: "500.00", step: 0.01, min: 0.01, required: false, disabled: !(@current_user.admin? || @current_user != @report.user) && !@report.draft?, data: { controller: "truncate-decimal", action: "truncate-decimal#truncate blur->truncate-decimal#pad" } %> + <%= form.number_field :maximum_amount, + placeholder: "Optional", + step: 0.01, + min: 0.01, + required: false, + style: "border: 0; min-height: 0; max-width:100%!important", + disabled: !(@current_user.admin? || @current_user != @report.user) && !@report.draft?, + data: { + controller: "truncate-decimal", + action: "truncate-decimal#truncate blur->truncate-decimal#pad" + } %>
<% end %> - <% if @report.submitted? || @report.draft? %> -
- <%= form.label :event_id, "Organization", class: "mt2" %> - <%= form.select(:event_id, - admin_signed_in? ? - Event.not_demo_mode.reorder(Event::CUSTOM_SORT).map { |event| [event.name, event.id] } : - ((@report.event ? [@report.event] : []) + current_user.events.not_hidden.filter_demo_mode(false)).uniq.map { |event| [event.name, event.id] }, - { prompt: "Select an event…", required: true }) %> - <% unless admin_signed_in? %> - You can transfer a report to any organization you're a part of. - <% end %> -
- <% end %> - <% if organizer_signed_in? && (@report.submitted? || @report.draft?) && @report.event %> -
- <%= form.label :reviewer_id, "Assigned reviewer", class: "mt2" %> - <%= form.collection_select :reviewer_id, @report.event.managers, :id, :name, include_blank: "Anyone" %> -
- <% end %> -
- <%= form.submit "Update report", class: "left mt2" %> - +
+ <% if @report.submitted? || @report.draft? %> +
+ <%= form.label :event_id, "Organization" %> + <%= form.select(:event_id, + admin_signed_in? ? + Event.not_demo_mode.reorder(Event::CUSTOM_SORT).map { |event| [event.name, event.id] } : + ((@report.event ? [@report.event] : []) + current_user.events.not_hidden.filter_demo_mode(false)).uniq.map { |event| [event.name, event.id] }, + { prompt: "Select an event…", required: true }) %> + <% unless admin_signed_in? %> + You can transfer a report to any organization you're a part of. + <% end %> +
+ <% end %> + <% if organizer_signed_in? && (@report.submitted? || @report.draft?) && @report.event %> +
+ <%= form.label :reviewer_id, "Assigned reviewer" %> + <%= form.collection_select :reviewer_id, @report.event.managers, :id, :name, include_blank: "Anyone" %> +
+ <% end %> +
+
+ <% if current_user.admin? && !@report.draft? %> - <% admin_tool("right mt2") do %> - <%= button_to "Delete report", reimbursement_report_path(@report), class: "bg-error btn", method: :delete, type: "button" %> + <% admin_tool("") do %> + <%= button_to "Delete", reimbursement_report_path(@report), class: "bg-error btn", method: :delete, type: "button" %> <% end %> <% else %> - <%= button_to "Delete report", reimbursement_report_path(@report), class: "right mt2 bg-error btn", method: :delete, type: "button" %> + <%= button_to "Delete", reimbursement_report_path(@report), class: "bg-error btn", method: :delete, type: "button" %> <% end %>
<% end %>