From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from firstgate.proxmox.com (firstgate.proxmox.com [212.224.123.68]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits)) (No client certificate requested) by lists.proxmox.com (Postfix) with ESMTPS id 936DF64660 for ; Thu, 3 Mar 2022 15:12:17 +0100 (CET) Received: from firstgate.proxmox.com (localhost [127.0.0.1]) by firstgate.proxmox.com (Proxmox) with ESMTP id 8264132D2B for ; Thu, 3 Mar 2022 15:11:47 +0100 (CET) Received: from proxmox-new.maurer-it.com (proxmox-new.maurer-it.com [94.136.29.106]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits)) (No client certificate requested) by firstgate.proxmox.com (Proxmox) with ESMTPS id 0B13032D21 for ; Thu, 3 Mar 2022 15:11:47 +0100 (CET) Received: from proxmox-new.maurer-it.com (localhost.localdomain [127.0.0.1]) by proxmox-new.maurer-it.com (Proxmox) with ESMTP id D59B046E84 for ; Thu, 3 Mar 2022 15:11:46 +0100 (CET) From: Dominik Csapak To: pve-devel@lists.proxmox.com Date: Thu, 3 Mar 2022 15:11:45 +0100 Message-Id: <20220303141145.352761-1-d.csapak@proxmox.com> X-Mailer: git-send-email 2.30.2 MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-SPAM-LEVEL: Spam detection results: 0 AWL 0.154 Adjusted score from AWL reputation of From: address BAYES_00 -1.9 Bayes spam probability is 0 to 1% KAM_DMARC_STATUS 0.01 Test Rule for DKIM or SPF Failure with Strict Alignment SPF_HELO_NONE 0.001 SPF: HELO does not publish an SPF Record SPF_PASS -0.001 SPF: sender matches SPF record T_SCC_BODY_TEXT_LINE -0.01 - URIBL_BLOCKED 0.001 ADMINISTRATOR NOTICE: The query to URIBL was blocked. See http://wiki.apache.org/spamassassin/DnsBlocklists#dnsbl-block for more information. [data.total] Subject: [pve-devel] [PATCH widget-toolkit] fix #3919: show first task output line correctly X-BeenThere: pve-devel@lists.proxmox.com X-Mailman-Version: 2.1.29 Precedence: list List-Id: Proxmox VE development discussion List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 03 Mar 2022 14:12:17 -0000 if a task did not produce output yet, we always get a single line with "no output". our heuristic in the gui counts the total lines + the current position. so to update the first output correctly, we have to update every time in case we only have one line. Otherwise, we only update on the second line, which is bad in case the only line we ever get is the result. Signed-off-by: Dominik Csapak --- src/panel/LogView.js | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/src/panel/LogView.js b/src/panel/LogView.js index 22029a1..412b26e 100644 --- a/src/panel/LogView.js +++ b/src/panel/LogView.js @@ -53,7 +53,12 @@ Ext.define('Proxmox.panel.LogView', { let data = viewModel.get('data'); if (first === data.first && total === data.total && lines.length === data.lines) { - return; // same content, skip setting and scrolling + // before there is any real output, we get 'no output' as a single line. + // so always update if we only have one to be sure to catch + // the first real line of output + if (total !== 1) { + return; // same content, skip setting and scrolling + } } viewModel.set('data', { first: first, -- 2.30.2