Dismiss context menu when scrolling (MozillaReality#2271)

lissyx · Jan 21, 2020 · 7fa1e4b · 7fa1e4b
1 parent 9c37fc0
commit 7fa1e4b
Show file tree

Hide file tree

Showing 5 changed files with 14 additions and 0 deletions.
diff --git a/app/src/common/shared/org/mozilla/vrbrowser/ui/callbacks/BookmarksCallback.java b/app/src/common/shared/org/mozilla/vrbrowser/ui/callbacks/BookmarksCallback.java
@@ -12,4 +12,5 @@ default void onSyncBookmarks(@NonNull View view) {}
     default void onFxALogin(@NonNull View view) {}
     default void onFxASynSettings(@NonNull View view) {}
     default void onShowContextMenu(@NonNull View view, Bookmark item, boolean isLastVisibleItem) {}
+    default void onHideContextMenu(@NonNull View view) {}
 }
diff --git a/app/src/common/shared/org/mozilla/vrbrowser/ui/callbacks/HistoryCallback.java b/app/src/common/shared/org/mozilla/vrbrowser/ui/callbacks/HistoryCallback.java
@@ -12,4 +12,5 @@ default void onSyncHistory(@NonNull View view) {}
     default void onFxALogin(@NonNull View view) {}
     default void onFxASynSettings(@NonNull View view) {}
     default void onShowContextMenu(@NonNull View view, @NonNull VisitInfo item, boolean isLastVisibleItem) {}
+    default void onHideContextMenu(@NonNull View view) {}
 }
diff --git a/app/src/common/shared/org/mozilla/vrbrowser/ui/views/BookmarksView.java b/app/src/common/shared/org/mozilla/vrbrowser/ui/views/BookmarksView.java
@@ -97,6 +97,7 @@ private void initialize(Context aContext) {
             v.requestFocusFromTouch();
             return false;
         });
+        mBinding.bookmarksList.setOnScrollChangeListener((v, scrollX, scrollY, oldScrollX, oldScrollY) -> mBookmarksViewListeners.forEach((listener) -> listener.onHideContextMenu(v)));
         mBinding.bookmarksList.setHasFixedSize(true);
         mBinding.bookmarksList.setItemViewCacheSize(20);
         mBinding.bookmarksList.setDrawingCacheEnabled(true);

diff --git a/app/src/common/shared/org/mozilla/vrbrowser/ui/views/HistoryView.java b/app/src/common/shared/org/mozilla/vrbrowser/ui/views/HistoryView.java
@@ -102,6 +102,7 @@ private void initialize(Context aContext) {
             v.requestFocusFromTouch();
             return false;
         });
+        mBinding.historyList.setOnScrollChangeListener((v, scrollX, scrollY, oldScrollX, oldScrollY) -> mHistoryViewListeners.forEach((listener) -> listener.onHideContextMenu(v)));
         mBinding.historyList.setHasFixedSize(true);
         mBinding.historyList.setItemViewCacheSize(20);
         mBinding.historyList.setDrawingCacheEnabled(true);

diff --git a/app/src/common/shared/org/mozilla/vrbrowser/ui/widgets/WindowWidget.java b/app/src/common/shared/org/mozilla/vrbrowser/ui/widgets/WindowWidget.java
@@ -1390,6 +1390,11 @@ public void onShowContextMenu(@NonNull View view, @NotNull Bookmark item, boolea
         public void onFxASynSettings(@NonNull View view) {
             mWidgetManager.getTray().toggleSettingsDialog(SettingsWidget.SettingDialog.FXA);
         }
+
+        @Override
+        public void onHideContextMenu(@NonNull View view) {
+            hideContextMenus();
+        }
     };
 
     private HistoryCallback mHistoryListener = new HistoryCallback() {
@@ -1414,6 +1419,11 @@ public void onShowContextMenu(@NonNull View view, @NonNull VisitInfo item, boole
         public void onFxASynSettings(@NonNull View view) {
             mWidgetManager.getTray().toggleSettingsDialog(SettingsWidget.SettingDialog.FXA);
         }
+
+        @Override
+        public void onHideContextMenu(@NonNull View view) {
+            hideContextMenus();
+        }
     };
 
     private void hideContextMenus() {