From 0263ec8cc863c669656dfb41df9ece8a710a676d Mon Sep 17 00:00:00 2001
From: "Guillaume (gdi)" <gdi@odoo.com>
Date: Thu, 20 Jan 2022 09:17:01 +0000
Subject: [PATCH] [IMP] website, website_sale, web_editor: set step on numeric
 inputs

Changing numeric values using the keyboard arrows did not work on some
fields. This commit solves this problem by adding the necessary
attributes to the numeric (we-)inputs.

task-2601533

Part-of: odoo/odoo#81241
---
 addons/web_editor/static/src/xml/wysiwyg.xml          | 2 +-
 addons/website/static/src/snippets/s_chart/options.js | 3 +++
 addons/website/views/snippets/s_chart.xml             | 4 ++--
 addons/website_sale/views/snippets/snippets.xml       | 2 +-
 4 files changed, 7 insertions(+), 4 deletions(-)

diff --git a/addons/web_editor/static/src/xml/wysiwyg.xml b/addons/web_editor/static/src/xml/wysiwyg.xml
index c64ad37a4e1..25ee17cb114 100644
--- a/addons/web_editor/static/src/xml/wysiwyg.xml
+++ b/addons/web_editor/static/src/xml/wysiwyg.xml
@@ -249,7 +249,7 @@
             <div>
                 <we-input class="o_we_user_value_widget" title="Border Width">
                     <div class="o_custom_border_width_input">
-                        <input name="custom_border_width" type="text" class="text-end" size="2"/>
+                        <input name="custom_border_width" type="number" class="text-end" size="2"/>
                         <span>px</span>
                     </div>
                 </we-input>
diff --git a/addons/website/static/src/snippets/s_chart/options.js b/addons/website/static/src/snippets/s_chart/options.js
index 5c9ca2b14f1..33cf1c95192 100644
--- a/addons/website/static/src/snippets/s_chart/options.js
+++ b/addons/website/static/src/snippets/s_chart/options.js
@@ -305,6 +305,9 @@ options.registry.InnerChart = options.Class.extend({
         const newEl = document.createElement(tag);
         const contentEl = document.createElement('input');
         contentEl.type = 'text';
+        if (tag === 'td') {
+            contentEl.type = 'number';
+        }
         contentEl.value = value || '';
         if (backgroundColor) {
             contentEl.dataset.backgroundColor = backgroundColor;
diff --git a/addons/website/views/snippets/s_chart.xml b/addons/website/views/snippets/s_chart.xml
index 379d8552587..80c0783cb03 100644
--- a/addons/website/views/snippets/s_chart.xml
+++ b/addons/website/views/snippets/s_chart.xml
@@ -53,8 +53,8 @@
                     </tr>
                 </table>
             </we-matrix>
-            <we-input string="Min Axis" data-select-data-attribute="" data-dependencies="bar_chart_opt, horizontal_bar_chart_opt, line_chart_opt" data-attribute-name="minValue"/>
-            <we-input string="Max Axis" data-select-data-attribute="" data-dependencies="bar_chart_opt, horizontal_bar_chart_opt, line_chart_opt" data-attribute-name="maxValue"/>
+            <we-input string="Min Axis" data-step="1" data-select-data-attribute="" data-dependencies="bar_chart_opt, horizontal_bar_chart_opt, line_chart_opt" data-attribute-name="minValue"/>
+            <we-input string="Max Axis" data-step="1" data-select-data-attribute="" data-dependencies="bar_chart_opt, horizontal_bar_chart_opt, line_chart_opt" data-attribute-name="maxValue"/>
             <we-colorpicker string="Background" data-name="chart_bg_color_opt"
                 data-color-change=""
                 data-attribute-name="backgroundColor"
diff --git a/addons/website_sale/views/snippets/snippets.xml b/addons/website_sale/views/snippets/snippets.xml
index 8ac3e453b38..84b060e0f37 100644
--- a/addons/website_sale/views/snippets/snippets.xml
+++ b/addons/website_sale/views/snippets/snippets.xml
@@ -20,7 +20,7 @@
                 <we-button data-customize-website-views="website_sale.products_list_view">List</we-button>
             </we-select>
             <we-row string="Size" class="o_we_sublevel_1">
-                <we-input data-set-ppg="" data-no-preview="true" data-reload="/"/>
+                <we-input data-set-ppg="" data-step="1" data-no-preview="true" data-reload="/"/>
                 <span class="mx-2 o_wsale_ppr_by">by</span>
                 <we-select class="o_wsale_ppr_submenu" data-dependencies="grid_view_opt" data-no-preview="true" data-reload="/">
                     <we-button data-set-ppr="2">2</we-button>