diff options
author | hyunyoungs <hyunyoungs@google.com> | 2022-06-07 22:12:33 -0700 |
---|---|---|
committer | hyunyoungs <hyunyoungs@google.com> | 2022-06-09 15:04:37 -0700 |
commit | e5f265c76ea21cf59187cf9acf6e8d94e7b6bb34 (patch) | |
tree | b1b5f42646a5db9a72f4d6c6ab20336ebacbd0d0 | |
parent | 2d5bda75fc7440373860411469c40192f5226b34 (diff) | |
download | Launcher3-e5f265c76ea21cf59187cf9acf6e8d94e7b6bb34.tar.gz |
Workspace should not receive touch events
Bug: 234648831
Test: touch outside recycler view inside container, doesn't swipe left/right
TL;DR;;
As the definition of ActiveRecyclerView changed in latest refactor,
scrollbar existence is no longer a condition to consume
touch event if user is in isSearching state.
Change-Id: Iec97fb036ad45796a77c59d35f329744398d4953
-rw-r--r-- | src/com/android/launcher3/allapps/BaseAllAppsContainerView.java | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/src/com/android/launcher3/allapps/BaseAllAppsContainerView.java b/src/com/android/launcher3/allapps/BaseAllAppsContainerView.java index 2c04fc7753..e90d4a6ba3 100644 --- a/src/com/android/launcher3/allapps/BaseAllAppsContainerView.java +++ b/src/com/android/launcher3/allapps/BaseAllAppsContainerView.java @@ -301,6 +301,10 @@ public abstract class BaseAllAppsContainerView<T extends Context & ActivityConte mTouchHandler.handleTouchEvent(ev, mFastScrollerOffset); return true; } + if (isSearching()) { + // if in search state, consume touch event. + return true; + } return false; } |