From 24f9531192af16c1b7fbb4e3dae5fd703843c97b Mon Sep 17 00:00:00 2001 From: David Vandorpe Date: Wed, 8 Jul 2015 13:37:19 +0200 Subject: [PATCH] Senpai Silox Styling Suggestions --- app/assets/javascripts/orders.js.coffee | 6 +++--- app/views/order_items/_order_item.html.erb | 2 +- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/app/assets/javascripts/orders.js.coffee b/app/assets/javascripts/orders.js.coffee index d3786eb..64c204f 100644 --- a/app/assets/javascripts/orders.js.coffee +++ b/app/assets/javascripts/orders.js.coffee @@ -20,7 +20,7 @@ ready = -> # Validate input, and then update -updateInput = (row, useRecalculate = true)-> +updateInput = (row, useRecalculate = true) -> cell = row.querySelector("input") if ! cell.validity.valid if(parseInt(cell.value) > parseInt(cell.max)) @@ -50,8 +50,8 @@ increment = (button, n) -> # Fix the counter counter = $(row).find('.row_counter') value = parseInt(counter.val()) - # Apparently CoffeeScript doesn't support ?: - value = if isNaN(value) then 0 else value + if isNaN(value) + value = 0 counter.val(value + n) updateInput(row[0]) diff --git a/app/views/order_items/_order_item.html.erb b/app/views/order_items/_order_item.html.erb index ddc2708..1027ec4 100644 --- a/app/views/order_items/_order_item.html.erb +++ b/app/views/order_items/_order_item.html.erb @@ -9,7 +9,7 @@ <%= content_tag :span, product.name %> <%= content_tag :small, euro(product.price) %> - <%= f.counter :count, max: product.stock, min:0, skip_label: true, wrapper_class: "input-group", class: "row_counter" %> + <%= f.counter :count, min: 0, max: product.stock, skip_label: true, wrapper_class: "input-group", class: "row_counter" %> <%= f.fields_for :product do |product| %> <%= product.hidden_field :price_cents, class: :price %> <%= product.hidden_field :stock, class: :stock %>