public inbox for pve-devel@lists.proxmox.com
 help / color / mirror / Atom feed
* [pve-devel] [PATCH widget-toolkit] file browser: show "Download" button by default and disable both initially
@ 2022-05-17 12:20 Fabian Ebner
  0 siblings, 0 replies; only message in thread
From: Fabian Ebner @ 2022-05-17 12:20 UTC (permalink / raw)
  To: pve-devel

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@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





^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2022-05-17 12:20 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-05-17 12:20 [pve-devel] [PATCH widget-toolkit] file browser: show "Download" button by default and disable both initially Fabian Ebner

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
Service provided by Proxmox Server Solutions GmbH | Privacy | Legal