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

Thomas Lamprecht t.lamprecht at proxmox.com
Mon Aug 23 18:17:26 CEST 2021


On 05/08/2021 12:31, Dominik Csapak wrote:
> 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(+)
> 
>

applied, thanks!





More information about the pve-devel mailing list