From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from gate001.proxmox.com (gate001.proxmox.com [45.144.208.40]) by lore.proxmox.com (Postfix) with ESMTPS id DFA691FF0C5 for ; Fri, 28 Aug 2026 10:00:30 +0200 (CEST) Received: from gate001.proxmox.com (localhost.localdomain [127.0.0.1]) by gate001.proxmox.com (Proxmox) with ESMTP id 81B562142C; Fri, 28 Aug 2026 10:00:30 +0200 (CEST) Message-ID: Date: Fri, 28 Aug 2026 10:00:25 +0200 MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH widget-toolkit v2] fix #7721: fix zoom interaction on charts with no initial store To: Dominik Csapak , pbs-devel@lists.proxmox.com References: <20260811085723.1272853-1-d.csapak@proxmox.com> Content-Language: en-US, de-DE From: Christian Ebner In-Reply-To: <20260811085723.1272853-1-d.csapak@proxmox.com> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit X-Bm-Milter-Handled: 55990f41-d878-4baa-be0a-ee34c49e34d2 X-Bm-Transport-Timestamp: 1787904016211 X-SPAM-LEVEL: Spam detection results: 0 AWL 0.708 Adjusted score from AWL reputation of From: address DMARC_MISSING 0.1 Missing DMARC policy KAM_DMARC_STATUS 0.01 Test Rule for DKIM or SPF Failure with Strict Alignment (newer systems) RCVD_IN_DNSWL_MED -2.3 Sender listed at https://www.dnswl.org/, medium trust SPF_HELO_NONE 0.001 SPF: HELO does not publish an SPF Record SPF_PASS -0.001 SPF: sender matches SPF record Message-ID-Hash: YOID6XNNQQ7IGJXDHTK5I2QNJOQNGADX X-Message-ID-Hash: YOID6XNNQQ7IGJXDHTK5I2QNJOQNGADX X-MailFrom: c.ebner@proxmox.com X-Mailman-Rule-Misses: dmarc-mitigation; no-senders; approved; loop; banned-address; emergency; member-moderation; nonmember-moderation; administrivia; implicit-dest; max-recipients; max-size; news-moderation; no-subject; digests; suspicious-header X-Mailman-Version: 3.3.10 Precedence: list List-Id: Proxmox Backup Server development discussion List-Help: List-Owner: List-Post: List-Subscribe: List-Unsubscribe: On 8/11/26 10:57 AM, Dominik Csapak wrote: > In our current ExtJs version (7.0), having animations disabled on charts > breaks the zoom interaction. One can zoom in and out once, but never > again. > > We generally keep animations enabled on our proxmoxRRDChart, but only > after the store's first load, to avoid animating from the empty store to > the initial data. This happens in initComponent on the store that was > given during instantiation. > > In some cases, however (e.g. the PBS datastore summary panel), the chart > has no initial store and instead gets one via 'setStore'. Here, the > event of the old (non-existent) store was never fired, so animations > were never activated and the zoom interaction was buggy. > > Fix this by overriding `setStore` to install an event handler for the > new store that enables the animations. If this runs more than once, it > would just set the animation value to the same again, which is fine. > > Also guard the delay method against calls from `setStore(null)` which > can happen when a component is being destroyed. > > Signed-off-by: Dominik Csapak > --- Double checked the animations already fixed in v1 for PBS, that the issue when switching between datastores is no longer present in v2 and that there are no regressions for PVE. Reviewed-by: Christian Ebner Tested-by: Christian Ebner