From 2c384bade859de28ba7172d51cf6104bfd9db25f Mon Sep 17 00:00:00 2001 From: Daniel Xie Date: Mon, 3 Jul 2017 22:16:41 -0500 Subject: [PATCH] Fixed bug with Firefox and deleting qty field in WSE --- src/StockMarket.js | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/src/StockMarket.js b/src/StockMarket.js index af1b1e8cc..674beb36f 100644 --- a/src/StockMarket.js +++ b/src/StockMarket.js @@ -262,6 +262,7 @@ function sellStock(stock, shares) { return false; } if (shares > stock.playerShares) {shares = stock.playerShares;} + if (shares == 0) {return false;} var gains = stock.price * shares - CONSTANTS.StockMarketCommission; Player.gainMoney(gains); stock.playerShares -= shares; @@ -437,7 +438,12 @@ function displayStockMarketContent() { qtyInput.setAttribute("type", "text"); qtyInput.setAttribute("id", tickerId + "-qty-input"); qtyInput.setAttribute("class", "stock-market-qty-input"); - qtyInput.setAttribute("onkeypress", 'return event.charCode >= 48 && event.charCode <= 57'); + qtyInput.setAttribute("onkeydown", "return ( event.ctrlKey || event.altKey " + + " || (4734 && event.keyCode<40) " + + " || (event.keyCode==46) )"); qtyInput.style.width = "3%"; qtyInput.style.display = "inline-block";