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

Commit 4f208ff3 authored Mar 16, 2020 by Selim Cinek

Fixed the double tap behavior on the expand button

Previously double taps would always open the notification
instead of the underlying app.

Fixes: 150705063
Test: add notification, double tap on expand button
Change-Id: I7993fde33695ff1a95bf4b4a923074989c4d3c60

parent 8ef86a12

packages/SystemUI/src/com/android/systemui/statusbar/notification/row/ActivatableNotificationView.java

+4 −0

Original line number	Diff line number	Diff line
		@@ -271,6 +271,10 @@ public abstract class ActivatableNotificationView extends ExpandableOutlineView
		}
		}

		public boolean isActive() {
		return mActivated;
		}

		private void startActivateAnimation(final boolean reverse) {
		if (!isAttachedToWindow()) {
		return;

packages/SystemUI/src/com/android/systemui/statusbar/notification/row/ActivatableNotificationViewController.java

+1 −1

Original line number	Diff line number	Diff line
		@@ -111,7 +111,7 @@ public class ActivatableNotificationViewController {
		if (mNeedsDimming && ev.getActionMasked() == MotionEvent.ACTION_DOWN
		&& mView.disallowSingleClick(ev)
		&& !mAccessibilityManager.isTouchExplorationEnabled()) {
		if (!mView.isActivated()) {
		if (!mView.isActive()) {
		return true;
		} else if (!mDoubleTapHelper.isWithinDoubleTapSlop(ev)) {
		mBlockNextTouch = true;