From 8a63a79896b3c22f94562ddc0aec1b939103494b Mon Sep 17 00:00:00 2001
From: Diego Prado Gesto <d.pradogesto@ultimaker.com>
Date: Sun, 11 Nov 2018 13:17:59 +0100
Subject: [PATCH] Add soft animation to the entire popup and not only in the
 background.

This prevents the content to show before the background.

Contributes to CURA-5786.
---
 .../ActionPanel/PrintInformationWidget.qml    | 38 ++++++++++---------
 1 file changed, 20 insertions(+), 18 deletions(-)

diff --git a/resources/qml/ActionPanel/PrintInformationWidget.qml b/resources/qml/ActionPanel/PrintInformationWidget.qml
index 620c3a408c..82707576e0 100644
--- a/resources/qml/ActionPanel/PrintInformationWidget.qml
+++ b/resources/qml/ActionPanel/PrintInformationWidget.qml
@@ -7,29 +7,30 @@ import QtQuick.Controls 2.1
 import UM 1.1 as UM
 import Cura 1.0 as Cura
 
-Button
+UM.RecolorImage
 {
     id: widget
 
-    implicitHeight: UM.Theme.getSize("section_icon").height
-    implicitWidth: UM.Theme.getSize("section_icon").width
+    //implicitHeight: UM.Theme.getSize("section_icon").height
+    //implicitWidth: UM.Theme.getSize("section_icon").width
 
-    background: UM.RecolorImage
+    source: UM.Theme.getIcon("info")
+    width: UM.Theme.getSize("section_icon").width
+    height: UM.Theme.getSize("section_icon").height
+
+    sourceSize.width: width
+    sourceSize.height: height
+
+    color: popup.opened ? UM.Theme.getColor("primary") : UM.Theme.getColor("text_medium")
+
+    MouseArea
     {
-        id: moreInformationIcon
-
-        source: UM.Theme.getIcon("info")
-        width: UM.Theme.getSize("section_icon").width
-        height: UM.Theme.getSize("section_icon").height
-
-        sourceSize.width: width
-        sourceSize.height: height
-
-        color: widget.hovered ? UM.Theme.getColor("primary") : UM.Theme.getColor("text_medium")
+        anchors.fill: parent
+        hoverEnabled: true
+        onEntered: popup.open()
+        onExited: popup.close()
     }
 
-    onClicked: popup.opened ? popup.close() : popup.open()
-
     Popup
     {
         id: popup
@@ -39,6 +40,9 @@ Button
 
         closePolicy: Popup.CloseOnEscape | Popup.CloseOnPressOutsideParent
 
+        opacity: opened ? 1 : 0
+        Behavior on opacity { NumberAnimation { duration: 100 } }
+
         contentItem: PrintJobInformation
         {
             id: printJobInformation
@@ -47,8 +51,6 @@ Button
 
         background: UM.PointingRectangle
         {
-            opacity: visible ? 1 : 0
-            Behavior on opacity { NumberAnimation { duration: 100 } }
             color: UM.Theme.getColor("tool_panel_background")
             borderColor: UM.Theme.getColor("lining")
             borderWidth: UM.Theme.getSize("default_lining").width