From aa0167be9d9a45ca411e85016400aa8850fffa13 Mon Sep 17 00:00:00 2001
From: Guillaume De Saint Martin <guillaumemdsm@gmail.com>
Date: Fri, 15 Dec 2023 12:18:16 +0100
Subject: [PATCH 1/3] [WebInterface] update dashboard margin

---
 Services/Interfaces/web_interface/templates/index.html | 9 +++++----
 1 file changed, 5 insertions(+), 4 deletions(-)

diff --git a/Services/Interfaces/web_interface/templates/index.html b/Services/Interfaces/web_interface/templates/index.html
index 91847fd58..df0477381 100644
--- a/Services/Interfaces/web_interface/templates/index.html
+++ b/Services/Interfaces/web_interface/templates/index.html
@@ -1,9 +1,10 @@
 {% extends "layout.html" %}
 {% set active_page = "home" %}
 {% block body %}
+    <br>
     <div update-url="{{ url_for('api.announcements') }}" class="alert alert-danger text-center d-none" role="alert" id="annoncementsAlert"></div>
     {% if not IS_CLOUD %}
-    <div class="d-none alert alert-success text-center mt-2" role="alert">
+    <div class="d-none alert alert-success text-center my-2" role="alert">
         <h5 class="d-none d-sm-inline"><span class="d-none d-md-inline"><i class="far fa-bell"></i> Good news ! </span>OctoBot version <span update-url="{{ url_for('api.upgrade_version') }}" id="upgradeVersion"></span> is available.</h5><button route="{{ url_for('commands', cmd='update') }}" type="button" class="btn btn-warning waves-effect">Upgrade now <i class="fas fa-cloud-download-alt"></i></button>
     </div>
     {% endif %}
@@ -17,7 +18,7 @@ <h5 class="d-none d-sm-inline"><span class="d-none d-md-inline"><i class="far fa
             </div>
         {% endif %}
         {% if is_in_stating_community_env() %}
-        <div class="card mt-2 mt-md-4">
+        <div class="card my-2 my-md-4">
             <div class="card-header"><h2>
                 Welcome to the OctoBot beta environment
                 </h2>
@@ -36,7 +37,7 @@ <h5 class="d-none d-sm-inline"><span class="d-none d-md-inline"><i class="far fa
             </div>
         </div>
         {% endif %}
-        <div class="card mt-2 mt-md-4" id="profitability-display">
+        <div class="card" id="profitability-display">
             <div class="card-header">
                 <h4>
                     <div class="row">
@@ -78,7 +79,7 @@ <h4>
                     <div id="portfolio_historyChart"
                          data-url="{{url_for('api.historical_portfolio_value', currency=reference_unit, time_frame='')}}"
                          data-reference-market="{{reference_unit}}"
-                         class="h-100 w-100">
+                         class="w-100">
                     </div>
                 </div>
             </div>

From 9a0394e36a0348edbd92118402fed2139852ba98 Mon Sep 17 00:00:00 2001
From: Guillaume De Saint Martin <guillaumemdsm@gmail.com>
Date: Fri, 15 Dec 2023 12:20:08 +0100
Subject: [PATCH 2/3] [WebInterface] reduce balance history height on smaller
 screens

---
 Services/Interfaces/web_interface/static/js/common/cst.js     | 1 +
 .../web_interface/static/js/common/custom_elements.js         | 3 ++-
 .../web_interface/static/js/common/portfolio_history.js       | 3 ++-
 Services/Interfaces/web_interface/static/js/common/util.js    | 4 ++++
 4 files changed, 9 insertions(+), 2 deletions(-)

diff --git a/Services/Interfaces/web_interface/static/js/common/cst.js b/Services/Interfaces/web_interface/static/js/common/cst.js
index 79ea8a5fc..c8a4cd21c 100644
--- a/Services/Interfaces/web_interface/static/js/common/cst.js
+++ b/Services/Interfaces/web_interface/static/js/common/cst.js
@@ -70,6 +70,7 @@ const profitability_update_interval = 5000;
 const portfolio_update_interval = 60000;
 
 const mobile_width_breakpoint = 1024;
+const medium_width_breakpoint = 1400;
 
 const material_colors = ["#2962ff", "#00b8d4", "#00c853", "#aeea00", "#ffab00",
     "#dd2c00", "#d50000", "#6200ea"];
diff --git a/Services/Interfaces/web_interface/static/js/common/custom_elements.js b/Services/Interfaces/web_interface/static/js/common/custom_elements.js
index 57e5be420..528c98066 100644
--- a/Services/Interfaces/web_interface/static/js/common/custom_elements.js
+++ b/Services/Interfaces/web_interface/static/js/common/custom_elements.js
@@ -93,7 +93,7 @@ function create_doughnut_chart(element, data, title, fontColor='white', animate=
     });
 }
 
-function create_line_chart(element, data, title, fontColor='white', update=true){
+function create_line_chart(element, data, title, fontColor='white', update=true, height=undefined){
     const trace = {
       x: data.map((e) => new Date(e.time*1000)),
       y: data.map((e) => e.value),
@@ -105,6 +105,7 @@ function create_line_chart(element, data, title, fontColor='white', update=true)
     const minDisplayY = Math.max(0, minY - ((maxDisplayY - minY) / 2));
     const layout = {
         title: title,
+        height: height,
         dragmode: isMobileDisplay() ? false : 'zoom',
         margin: {
             l: 30,
diff --git a/Services/Interfaces/web_interface/static/js/common/portfolio_history.js b/Services/Interfaces/web_interface/static/js/common/portfolio_history.js
index 5740a91d3..5ad1e0faf 100644
--- a/Services/Interfaces/web_interface/static/js/common/portfolio_history.js
+++ b/Services/Interfaces/web_interface/static/js/common/portfolio_history.js
@@ -24,10 +24,11 @@ $(document).ready(function() {
         const success = (updated_data, update_url, dom_root_element, msg, status) => {
             const graphDiv = $(`#profitability_graph`);
             const defaultDiv = $(`#no_profitability_graph`);
+            const height = isMobileDisplay()? 250 : isMediumDisplay() ? 450 : undefined;
             if(msg.length > 1){
                 graphDiv.removeClass(hidden_class);
                 defaultDiv.addClass(hidden_class);
-                create_line_chart(document.getElementById(element_id), msg, title, 'white', update);
+                create_line_chart(document.getElementById(element_id), msg, title, 'white', update, height);
             }else{
                 graphDiv.addClass(hidden_class);
                 defaultDiv.removeClass(hidden_class);
diff --git a/Services/Interfaces/web_interface/static/js/common/util.js b/Services/Interfaces/web_interface/static/js/common/util.js
index c0cf849f5..835e4c806 100644
--- a/Services/Interfaces/web_interface/static/js/common/util.js
+++ b/Services/Interfaces/web_interface/static/js/common/util.js
@@ -147,6 +147,10 @@ function isMobileDisplay() {
     return $(window).width() < mobile_width_breakpoint;
 }
 
+function isMediumDisplay() {
+    return $(window).width() < medium_width_breakpoint;
+}
+
 const handle_rounded_numbers_display = () => {
     $(".rounded-number").each(function (){
         const text = $(this).text().trim();

From d953ed719189117d18a279540df0582af743ad1a Mon Sep 17 00:00:00 2001
From: Guillaume De Saint Martin <guillaumemdsm@gmail.com>
Date: Fri, 15 Dec 2023 16:45:18 +0100
Subject: [PATCH 3/3] [Exchanges] use factorized contract size handling for
 orders

---
 Trading/Exchange/binance/binance_exchange.py | 6 ------
 Trading/Exchange/kucoin/kucoin_exchange.py   | 9 ++-------
 Trading/Exchange/okx/okx_exchange.py         | 6 ------
 3 files changed, 2 insertions(+), 19 deletions(-)

diff --git a/Trading/Exchange/binance/binance_exchange.py b/Trading/Exchange/binance/binance_exchange.py
index d30c344c1..85f5b4dc7 100644
--- a/Trading/Exchange/binance/binance_exchange.py
+++ b/Trading/Exchange/binance/binance_exchange.py
@@ -221,12 +221,6 @@ class BinanceCCXTAdapter(exchanges.CCXTAdapter):
 
     def fix_order(self, raw, symbol=None, **kwargs):
         fixed = super().fix_order(raw, **kwargs)
-        if self.connector.exchange_manager.is_future \
-                and fixed[trading_enums.ExchangeConstantsOrderColumns.AMOUNT.value] is not None:
-            # amount is in contact, multiply by contract value to get the currency amount (displayed to the user)
-            contract_size = self.connector.get_contract_size(symbol)
-            fixed[trading_enums.ExchangeConstantsOrderColumns.AMOUNT.value] = \
-                fixed[trading_enums.ExchangeConstantsOrderColumns.AMOUNT.value] * float(contract_size)
         self._adapt_order_type(fixed)
         return fixed
 
diff --git a/Trading/Exchange/kucoin/kucoin_exchange.py b/Trading/Exchange/kucoin/kucoin_exchange.py
index 97723b687..a69f8abc5 100644
--- a/Trading/Exchange/kucoin/kucoin_exchange.py
+++ b/Trading/Exchange/kucoin/kucoin_exchange.py
@@ -384,13 +384,8 @@ class KucoinCCXTAdapter(exchanges.CCXTAdapter):
     def fix_order(self, raw, symbol=None, **kwargs):
         raw_order_info = raw[ccxt_enums.ExchangePositionCCXTColumns.INFO.value]
         fixed = super().fix_order(raw, **kwargs)
-        if self.connector.exchange_manager.is_future \
-                and fixed[trading_enums.ExchangeConstantsOrderColumns.AMOUNT.value] is not None:
-            # amount is in contact, multiply by contract value to get the currency amount (displayed to the user)
-            contract_size = self.connector.get_contract_size(symbol)
-            fixed[trading_enums.ExchangeConstantsOrderColumns.AMOUNT.value] = \
-                fixed[trading_enums.ExchangeConstantsOrderColumns.AMOUNT.value] * float(contract_size)
-        if fixed[trading_enums.ExchangeConstantsOrderColumns.COST.value] is not None:
+        if self.connector.exchange_manager.is_future and \
+                fixed[trading_enums.ExchangeConstantsOrderColumns.COST.value] is not None:
             fixed[trading_enums.ExchangeConstantsOrderColumns.COST.value] = \
                 fixed[trading_enums.ExchangeConstantsOrderColumns.COST.value] * \
                 float(raw_order_info.get(self.KUCOIN_LEVERAGE, 1))
diff --git a/Trading/Exchange/okx/okx_exchange.py b/Trading/Exchange/okx/okx_exchange.py
index e11a2e7dc..6e6de8d2c 100644
--- a/Trading/Exchange/okx/okx_exchange.py
+++ b/Trading/Exchange/okx/okx_exchange.py
@@ -453,12 +453,6 @@ class OKXCCXTAdapter(exchanges.CCXTAdapter):
 
     def fix_order(self, raw, symbol=None, **kwargs):
         fixed = super().fix_order(raw, **kwargs)
-        if self.connector.exchange_manager.is_future \
-                and fixed[trading_enums.ExchangeConstantsOrderColumns.AMOUNT.value] is not None:
-            # amount is in contact, multiply by contract value to get the currency amount (displayed to the user)
-            contract_size = self.connector.get_contract_size(symbol)
-            fixed[trading_enums.ExchangeConstantsOrderColumns.AMOUNT.value] = \
-                fixed[trading_enums.ExchangeConstantsOrderColumns.AMOUNT.value] * float(contract_size)
         self._adapt_order_type(fixed)
         return fixed