From 5963c6ddc3c66d0237eeb21e31b22efe49d5fcbb Mon Sep 17 00:00:00 2001 From: "Anthony J. Thibault" Date: Fri, 17 Feb 2017 11:04:04 -0800 Subject: [PATCH] Fix for toolbar visibility after snapshots. --- .../resources/qml/hifi/toolbars/Toolbar.qml | 21 +++++++------------ 1 file changed, 7 insertions(+), 14 deletions(-) diff --git a/interface/resources/qml/hifi/toolbars/Toolbar.qml b/interface/resources/qml/hifi/toolbars/Toolbar.qml index f62e4dea19..0080e49815 100644 --- a/interface/resources/qml/hifi/toolbars/Toolbar.qml +++ b/interface/resources/qml/hifi/toolbars/Toolbar.qml @@ -25,7 +25,7 @@ Window { property real buttonSize: 50; property var buttons: [] property var container: horizontal ? row : column - + Settings { category: "toolbar/" + window.objectName property alias x: window.x @@ -49,6 +49,7 @@ Window { id: content implicitHeight: horizontal ? row.height : column.height implicitWidth: horizontal ? row.width : column.width + property bool wasVisibleBeforeBeingPinned: false Row { id: row @@ -65,19 +66,11 @@ Window { Connections { target: desktop onPinnedChanged: { - if (!window.pinned) { - return; - } - var newPinned = desktop.pinned; - for (var i in buttons) { - var child = buttons[i]; - if (desktop.pinned) { - if (!child.pinned) { - child.visible = false; - } - } else { - child.visible = true; - } + if (desktop.pinned) { + content.wasVisibleBeforeBeingPinned = window.visible; + window.visible = false; + } else { + window.visible = content.wasVisibleBeforeBeingPinned; } } }