Skip to content
This repository was archived by the owner on Jul 22, 2024. It is now read-only.

Commit dc5a0ec

Browse files
keianhzoMortimerGoro
authored andcommitted
Fix for disappearing context menus (#2376)
1 parent 1691b1d commit dc5a0ec

File tree

6 files changed

+13
-8
lines changed

6 files changed

+13
-8
lines changed

app/src/common/shared/org/mozilla/vrbrowser/ui/widgets/NavigationBarWidget.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -418,7 +418,7 @@ public void detachFromWindow() {
418418
}
419419
mAttachedWindow = null;
420420
if (mAwesomeBar != null && mAwesomeBar.isVisible()) {
421-
onHideAwesomeBar();
421+
mAwesomeBar.hideNoAnim(UIWidget.KEEP_WIDGET);
422422
}
423423
}
424424

app/src/common/shared/org/mozilla/vrbrowser/ui/widgets/SuggestionsWidget.java

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -24,6 +24,7 @@
2424
import org.mozilla.vrbrowser.R;
2525
import org.mozilla.vrbrowser.audio.AudioEngine;
2626
import org.mozilla.vrbrowser.ui.views.CustomListView;
27+
import org.mozilla.vrbrowser.utils.ViewUtils;
2728

2829
import java.util.ArrayList;
2930
import java.util.List;
@@ -125,11 +126,15 @@ public void hide(@HideFlags int aHideFlags) {
125126
mList.startAnimation(mScaleDownAnimation);
126127
}
127128

129+
public void hideNoAnim(@HideFlags int aHideFlags) {
130+
super.hide(aHideFlags);
131+
}
132+
128133
// FocusChangeListener
129134

130135
@Override
131136
public void onGlobalFocusChanged(View oldFocus, View newFocus) {
132-
if (oldFocus != null && isVisible()) {
137+
if (!ViewUtils.isEqualOrChildrenOf(this, newFocus)) {
133138
onDismiss();
134139
}
135140
}

app/src/common/shared/org/mozilla/vrbrowser/ui/widgets/dialogs/SelectionActionWidget.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -184,7 +184,7 @@ private void handleAction(View sender) {
184184

185185
@Override
186186
public void onGlobalFocusChanged(View oldFocus, View newFocus) {
187-
if (!ViewUtils.isChildrenOf(getChildAt(0), newFocus)) {
187+
if (!ViewUtils.isEqualOrChildrenOf(this, newFocus)) {
188188
onDismiss();
189189
}
190190
}

app/src/common/shared/org/mozilla/vrbrowser/ui/widgets/menus/ContextMenuWidget.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -122,7 +122,7 @@ public void setContextElement(GeckoSession.ContentDelegate.ContextElement aConte
122122

123123
@Override
124124
public void onGlobalFocusChanged(View oldFocus, View newFocus) {
125-
if (!ViewUtils.isChildrenOf(this, newFocus) && isVisible()) {
125+
if (!ViewUtils.isEqualOrChildrenOf(this, newFocus) && isVisible()) {
126126
onDismiss();
127127
}
128128
}

app/src/common/shared/org/mozilla/vrbrowser/ui/widgets/menus/HamburgerMenuWidget.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -123,8 +123,8 @@ private void createMenuItems() {
123123

124124
@Override
125125
public void onGlobalFocusChanged(View oldFocus, View newFocus) {
126-
if (!ViewUtils.isChildrenOf(menuContainer, newFocus)) {
127-
hide(KEEP_WIDGET);
126+
if (!ViewUtils.isEqualOrChildrenOf(this, newFocus)) {
127+
onDismiss();
128128
}
129129
}
130130
}

app/src/common/shared/org/mozilla/vrbrowser/ui/widgets/menus/LibraryMenuWidget.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -140,8 +140,8 @@ private void createMenuItems(boolean canOpenWindows, boolean isBookmarked) {
140140

141141
@Override
142142
public void onGlobalFocusChanged(View oldFocus, View newFocus) {
143-
if (!ViewUtils.isChildrenOf(menuContainer, newFocus)) {
144-
hide(KEEP_WIDGET);
143+
if (!ViewUtils.isEqualOrChildrenOf(this, newFocus)) {
144+
onDismiss();
145145
}
146146
}
147147
}

0 commit comments

Comments
 (0)