[pve-devel] [PATCH widget-toolkit] file browser: show "Download" button by default and disable both initially

Fabian Ebner f.ebner at proxmox.com
Tue May 17 14:20:22 CEST 2022


Previously, the "Download As" button/menu would show initially and
clicking it had no effect. In PVE, this was noticeable and confusing.
In PBS, the rootdir is already auto-selected when the store loads.
Since the button changes automatically with the selection, the issue
was not noticeable there.

Signed-off-by: Fabian Ebner <f.ebner at proxmox.com>
---
 src/window/FileBrowser.js | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/src/window/FileBrowser.js b/src/window/FileBrowser.js
index a519d6b..8db7c51 100644
--- a/src/window/FileBrowser.js
+++ b/src/window/FileBrowser.js
@@ -308,12 +308,13 @@ Ext.define("Proxmox.window.FileBrowser", {
 	    handler: 'downloadZip',
 	    reference: 'downloadBtn',
 	    disabled: true,
-	    hidden: true,
 	},
 	{
 	    text: gettext('Download as'),
 	    xtype: 'button',
 	    reference: 'menuBtn',
+	    disabled: true,
+	    hidden: true,
 	    menu: {
 		items: [
 		    {
-- 
2.30.2






More information about the pve-devel mailing list