Fix 'Private' HMD preview texture enable/disable...again

This commit is contained in:
Zach Fox 2018-04-03 13:46:02 -07:00
parent 987aa40783
commit f20eddefc8
7 changed files with 31 additions and 24 deletions

View file

@ -24,6 +24,18 @@ Item {
HifiConstants { id: hifi; } HifiConstants { id: hifi; }
id: root; id: root;
// This will cause a bug -- if you bring up passphrase selection in HUD mode while
// in HMD while having HMD preview enabled, then move, then finish passphrase selection,
// HMD preview will stay off.
// TODO: Fix this unlikely bug
onVisibleChanged: {
if (visible) {
sendSignalToWallet({method: 'disableHmdPreview'});
} else {
sendSignalToWallet({method: 'maybeEnableHmdPreview'});
}
}
// Username Text // Username Text
RalewayRegular { RalewayRegular {

View file

@ -68,10 +68,6 @@ Item {
propagateComposedEvents: false; propagateComposedEvents: false;
hoverEnabled: true; hoverEnabled: true;
} }
Component.onDestruction: {
sendSignalToParent({method: 'maybeEnableHmdPreview'});
}
// This will cause a bug -- if you bring up passphrase selection in HUD mode while // This will cause a bug -- if you bring up passphrase selection in HUD mode while
// in HMD while having HMD preview enabled, then move, then finish passphrase selection, // in HMD while having HMD preview enabled, then move, then finish passphrase selection,

View file

@ -61,9 +61,6 @@ Item {
if (root.shouldImmediatelyFocus) { if (root.shouldImmediatelyFocus) {
focusFirstTextField(); focusFirstTextField();
} }
sendMessageToLightbox({method: 'disableHmdPreview'});
} else {
sendMessageToLightbox({method: 'maybeEnableHmdPreview'});
} }
} }

View file

@ -44,6 +44,17 @@ Item {
} }
} }
// This will cause a bug -- if you bring up security image selection in HUD mode while
// in HMD while having HMD preview enabled, then move, then finish passphrase selection,
// HMD preview will stay off.
// TODO: Fix this unlikely bug
onVisibleChanged: {
if (visible) {
sendSignalToWallet({method: 'disableHmdPreview'});
} else {
sendSignalToWallet({method: 'maybeEnableHmdPreview'});
}
}
// Security Image // Security Image
Item { Item {

View file

@ -25,18 +25,6 @@ Item {
id: root; id: root;
property alias currentIndex: securityImageGrid.currentIndex; property alias currentIndex: securityImageGrid.currentIndex;
// This will cause a bug -- if you bring up security image selection in HUD mode while
// in HMD while having HMD preview enabled, then move, then finish passphrase selection,
// HMD preview will stay off.
// TODO: Fix this unlikely bug
onVisibleChanged: {
if (visible) {
sendSignalToWallet({method: 'disableHmdPreview'});
} else {
sendSignalToWallet({method: 'maybeEnableHmdPreview'});
}
}
SecurityImageModel { SecurityImageModel {
id: gridModel; id: gridModel;

View file

@ -237,7 +237,7 @@ Rectangle {
} else { } else {
sendToScript(msg); sendToScript(msg);
} }
} else if (msg.method === 'maybeEnableHmdPreview') { } else {
sendToScript(msg); sendToScript(msg);
} }
} }

View file

@ -76,6 +76,12 @@ Item {
var currentStepNumber = root.activeView.substring(5); var currentStepNumber = root.activeView.substring(5);
UserActivityLogger.commerceWalletSetupProgress(timestamp, root.setupAttemptID, UserActivityLogger.commerceWalletSetupProgress(timestamp, root.setupAttemptID,
Math.round((timestamp - root.startingTimestamp)/1000), currentStepNumber, root.setupStepNames[currentStepNumber - 1]); Math.round((timestamp - root.startingTimestamp)/1000), currentStepNumber, root.setupStepNames[currentStepNumber - 1]);
if (root.activeView === "step_2" || root.activeView === "step_3") {
sendSignalToWallet({method: 'disableHmdPreview'});
} else {
sendSignalToWallet({method: 'maybeEnableHmdPreview'});
}
} }
// //
@ -441,7 +447,7 @@ Item {
} }
Item { Item {
id: choosePassphraseContainer; id: choosePassphraseContainer;
visible: root.hasShownSecurityImageTip && root.activeView === "step_3"; visible: root.activeView === "step_3";
// Anchors // Anchors
anchors.top: titleBarContainer.bottom; anchors.top: titleBarContainer.bottom;
anchors.topMargin: 30; anchors.topMargin: 30;
@ -451,10 +457,7 @@ Item {
onVisibleChanged: { onVisibleChanged: {
if (visible) { if (visible) {
sendSignalToWallet({method: 'disableHmdPreview'});
Commerce.getWalletAuthenticatedStatus(); Commerce.getWalletAuthenticatedStatus();
} else {
sendSignalToWallet({method: 'maybeEnableHmdPreview'});
} }
} }