From eaf413997d0c5ca4b317a0d65b5aef5c44131627 Mon Sep 17 00:00:00 2001
From: Jaime van Kessel <nallath@gmail.com>
Date: Fri, 7 Dec 2018 09:44:09 +0100
Subject: [PATCH] Changed the installed button to be a secondary button

CURA-6006
---
 .../qml/ToolboxDetailTileActions.qml          | 40 ++++++++++++-------
 .../resources/qml/ToolboxProgressButton.qml   |  1 +
 2 files changed, 27 insertions(+), 14 deletions(-)

diff --git a/plugins/Toolbox/resources/qml/ToolboxDetailTileActions.qml b/plugins/Toolbox/resources/qml/ToolboxDetailTileActions.qml
index cc32ff032d..848acfbf4f 100644
--- a/plugins/Toolbox/resources/qml/ToolboxDetailTileActions.qml
+++ b/plugins/Toolbox/resources/qml/ToolboxDetailTileActions.qml
@@ -16,24 +16,36 @@ Column
     width: UM.Theme.getSize("toolbox_action_button").width
     spacing: UM.Theme.getSize("narrow_margin").height
 
-    ToolboxProgressButton
+    Item
     {
-        id: installButton
-        active: toolbox.isDownloading && toolbox.activePackage == model
-        complete: installed
-        onReadyAction:
+        width: installButton.width
+        height: installButton.height
+        ToolboxProgressButton
         {
-            toolbox.activePackage = model
-            toolbox.startDownload(model.download_url)
+            id: installButton
+            active: toolbox.isDownloading && toolbox.activePackage == model
+            onReadyAction:
+            {
+                toolbox.activePackage = model
+                toolbox.startDownload(model.download_url)
+            }
+            onActiveAction: toolbox.cancelDownload()
+
+            // Don't allow installing while another download is running
+            enabled: installed || (!(toolbox.isDownloading && toolbox.activePackage != model) && !loginRequired)
+            opacity: enabled ? 1.0 : 0.5
+            visible: !updateButton.visible && !installed// Don't show when the update button is visible
         }
-        onActiveAction: toolbox.cancelDownload()
 
-        onCompleteAction: toolbox.viewCategory = "installed"
-
-        // Don't allow installing while another download is running
-        enabled: installed || (!(toolbox.isDownloading && toolbox.activePackage != model) && !loginRequired)
-        opacity: enabled ? 1.0 : 0.5
-        visible: !updateButton.visible // Don't show when the update button is visible
+        Cura.SecondaryButton
+        {
+            visible: installed
+            onClicked: toolbox.viewCategory = "installed"
+            text: catalog.i18nc("@action:button", "Installed")
+            fixedWidthMode: true
+            width: installButton.width
+            height: installButton.height
+        }
     }
 
     Label
diff --git a/plugins/Toolbox/resources/qml/ToolboxProgressButton.qml b/plugins/Toolbox/resources/qml/ToolboxProgressButton.qml
index 0b574e8653..3ca18a52ed 100644
--- a/plugins/Toolbox/resources/qml/ToolboxProgressButton.qml
+++ b/plugins/Toolbox/resources/qml/ToolboxProgressButton.qml
@@ -7,6 +7,7 @@ import QtQuick.Controls.Styles 1.4
 import UM 1.1 as UM
 import Cura 1.0 as Cura
 
+// TODO; This is in quite some need for refactoring.
 Item
 {
     id: base