[pve-devel] [PATCH widget-toolkit] panel/RRDCharts: enable scrolling for RRDCharts on touchscreens

Dominik Csapak d.csapak at proxmox.com
Thu Aug 5 12:31:35 CEST 2021


quote from extjs docs (AbstractChart.js):

 If you do have a chart inside a scrollable view, even if it has no
 interactions, you have to set its touchAction config to the following:

 touchAction: {
     panX: true,
     panY: true
 }

 Otherwise, if a touch action started on a chart,
 a swipe will not scroll the view.

Since we have RRDCharts always in a scrollable view, and it does not seem to
hurt non-touchscreens, enable it by default so that users on touchscreens
can scroll it.

Reported by users in the forum:
https://forum.proxmox.com/threads/minor-but-annoying-ipad-swipe-bug.93686

Signed-off-by: Dominik Csapak <d.csapak at proxmox.com>
---
 src/panel/RRDChart.js | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/src/panel/RRDChart.js b/src/panel/RRDChart.js
index d63170e..41533ea 100644
--- a/src/panel/RRDChart.js
+++ b/src/panel/RRDChart.js
@@ -155,6 +155,11 @@ Ext.define('Proxmox.widget.RRDChart', {
 	},
     },
 
+    touchAction: {
+	panX: true,
+	panY: true,
+    },
+
     constructor: function(config) {
 	let me = this;
 
-- 
2.30.2






More information about the pve-devel mailing list