Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit a6024e86 authored by Ben Lin's avatar Ben Lin
Browse files

Fix accessibility regression. 

Instead of setting mHideMenuAfterShown to true for all cases, only do it
when accessibility isn't enabled.

Bug: 158662766
Test: Double tap when PIP focused
Change-Id: I18713ab1fdf511b74108df355204b2c54913a078
parent c4568caf

packages/SystemUI/src/com/android/systemui/pip/phone/PipTouchHandler.java

+1 −1
Original line number Diff line number Diff line
@@ -688,11 +688,11 @@ public class PipTouchHandler { 
                break;

            }

            case MotionEvent.ACTION_HOVER_EXIT: {

                mHideMenuAfterShown = true;

                // If Touch Exploration is enabled, some a11y services (e.g. Talkback) is probably

                // on and changing MotionEvents into HoverEvents.

                // Let's not enable menu show/hide for a11y services.

                if (!mAccessibilityManager.isTouchExplorationEnabled()) {

                    mHideMenuAfterShown = true;

                    mMenuController.hideMenu();

                }

                if (!shouldDeliverToMenu && mSendingHoverAccessibilityEvents) {