Improve invoice and supplier show views (#840)

app/assets/stylesheets/bootstrap_and_overrides.css.less

@@ -335,10 +335,9 @@ td.symbol, th.symbol {
 // https://github.com/twbs/bootstrap/issues/4550#issuecomment-31916049
 .dropdown-backdrop { position: static; }
 
-// need more space for supplier&order information (in German, at least)
 .dl-horizontal {
-  dt { width: 160px; }
-  dd { margin-left: 170px; }
+  dt { width: 200px; margin-bottom: 0.2em; }
+  dd { margin-left: 210px; }
 }
 
 // get rid of extra bottom margin inside well

app/views/finance/invoices/show.html.haml

@@ -2,30 +2,31 @@
 
 - total = 0
 
-%p
-  %b= heading_helper(Invoice, :created_at) + ':'
-  = format_time(@invoice.created_at)
-%p
-  %b= heading_helper(Invoice, :created_by) + ':'
-  = show_user(@invoice.created_by)
-%p
-  %b= heading_helper(Invoice, :supplier) + ':'
-  = @invoice.supplier.name
+.row-fluid
+  .span6
+    %dl.dl-horizontal
+      %dt= heading_helper(Invoice, :created_at) + ':'
+      %dd= format_time(@invoice.created_at)
 
-- if @invoice.deliveries.any?
-  %p
-    %b= heading_helper(Invoice, :deliveries) + ':'
-    %span><
+      %dt= heading_helper(Invoice, :created_by) + ':'
+      %dd= show_user(@invoice.created_by)
+
+      %dt= heading_helper(Invoice, :supplier) + ':'
+      %dd= @invoice.supplier.name
+
+      - if @invoice.deliveries.any?
+        %dt= heading_helper(Invoice, :deliveries) + ':'
+        %dd><
           - @invoice.deliveries.order(:date).each_with_index do |delivery, index|
             - sum = delivery.sum
             - total += sum
             = ', ' if index > 0
             = link_to format_date(delivery.date), [delivery.supplier, delivery]
             = ' (' + number_to_currency(sum) + ')'
-- if @invoice.orders.any?
-  %p
-    %b= heading_helper(Invoice, :orders) + ':'
-    %span><
+
+      - if @invoice.orders.any?
+        %dt= heading_helper(Invoice, :orders) + ':'
+        %dd><
           - @invoice.orders.order(:ends).each_with_index do |order, index|
             - sum = order.sum
             - total += sum
@@ -33,43 +34,44 @@
             = link_to format_date(order.ends), new_finance_order_path(order_id: order)
             = ' (' + number_to_currency(sum) + ')'
 
-%p
-  %b= heading_helper(Invoice, :number) + ':'
-  = @invoice.number
-%p
-  %b= heading_helper(Invoice, :date) + ':'
-  = @invoice.date
-%p
-  %b= heading_helper(Invoice, :paid_on) + ':'
-  = @invoice.paid_on
-%p
-  %b= heading_helper(Invoice, :amount) + ':'
-  = number_to_currency @invoice.amount
-%p
-  %b= heading_helper(Invoice, :deposit) + ':'
-  = number_to_currency @invoice.deposit
-%p
-  %b= heading_helper(Invoice, :deposit_credit) + ':'
-  = number_to_currency @invoice.deposit_credit
-%p
-  %b= heading_helper(Invoice, :net_amount) + ':'
-  = number_to_currency @invoice.net_amount
-- if @invoice.deliveries.any? || @invoice.orders.any?
-  %p
-    %b= heading_helper(Invoice, :total) + ':'
-    = number_to_currency total
-%p
-  %b= heading_helper(Invoice, :attachment) + ':'
-  - if @invoice.attachment_data
-    = link_to t('ui.download'), finance_invoice_attachment_path(@invoice)
-%p
-  %b= heading_helper(Invoice, :note) + ':'
-  =h @invoice.note
-- if @invoice.financial_link
-  %p
-    %b= heading_helper(Invoice, :financial_link) + ':'
-    = link_to t('ui.show'), finance_link_path(@invoice.financial_link)
+      %dt= heading_helper(Invoice, :number) + ':'
+      %dd= @invoice.number
 
-- if @invoice.user_can_edit?(current_user)
+      %dt= heading_helper(Invoice, :date) + ':'
+      %dd= format_date(@invoice.date)
+
+      %dt= heading_helper(Invoice, :paid_on) + ':'
+      %dd= format_date(@invoice.paid_on)
+
+      %dt= heading_helper(Invoice, :amount) + ':'
+      %dd= number_to_currency @invoice.amount
+
+      %dt= heading_helper(Invoice, :deposit) + ':'
+      %dd= number_to_currency @invoice.deposit
+
+      %dt= heading_helper(Invoice, :deposit_credit) + ':'
+      %dd= number_to_currency @invoice.deposit_credit
+
+      %dt= heading_helper(Invoice, :net_amount) + ':'
+      %dd= number_to_currency @invoice.net_amount
+
+      - if @invoice.deliveries.any? || @invoice.orders.any?
+        %dt= heading_helper(Invoice, :total) + ':'
+        %dd= number_to_currency total
+
+      - if @invoice.attachment_data
+        %dt= heading_helper(Invoice, :attachment) + ':'
+        %dd= link_to t('ui.download'), finance_invoice_attachment_path(@invoice)
+
+      %dt= heading_helper(Invoice, :note) + ':'
+      %dd= simple_format(@invoice.note)
+
+      - if @invoice.financial_link
+        %dt= heading_helper(Invoice, :financial_link) + ':'
+        %dd= link_to t('ui.show'), finance_link_path(@invoice.financial_link)
+
+    .clearfix
+      .form-actions
+        - if @invoice.user_can_edit?(current_user)
           = link_to t('ui.edit'), edit_finance_invoice_path(@invoice), class: 'btn'
-= link_to t('ui.or_cancel'), finance_invoices_path
+        = link_to t('ui.or_cancel'), finance_invoices_path

app/views/suppliers/show.html.haml

@@ -38,9 +38,9 @@
       %dt= heading_helper(Supplier, :delivery_days) + ':'
       %dd= @supplier.delivery_days
       %dt= heading_helper(Supplier, :order_howto) + ':'
-      %dd= @supplier.order_howto
+      %dd= simple_format(@supplier.order_howto)
       %dt= heading_helper(Supplier, :note) + ':'
-      %dd= @supplier.note
+      %dd= simple_format(@supplier.note)
       %dt= heading_helper(Supplier, :min_order_quantity) + ':'
       %dd= @supplier.min_order_quantity