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

Commit 18362e2c authored by alexmang's avatar alexmang
Browse files

Fixing a11y bubble issues

- activating manage focuses on first item in manage menu
- scrimview no longer highlighted expanded bubble focus list
- bubble now says double tap to activate

Bug: 158811290
Bug: 158811563
Bug: 158811388
Test: manual
Change-Id: I72f348d1407df6fe73103001145891d55b7537da
parent 80bf9c82
Loading
Loading
Loading
Loading
+1 −0
Original line number Original line Diff line number Diff line
@@ -100,6 +100,7 @@ public class BadgedImageView extends ImageView {
        mDotRenderer = new DotRenderer(mBubbleBitmapSize, iconPath, DEFAULT_PATH_SIZE);
        mDotRenderer = new DotRenderer(mBubbleBitmapSize, iconPath, DEFAULT_PATH_SIZE);


        setFocusable(true);
        setFocusable(true);
        setClickable(true);
    }
    }


    /**
    /**
+2 −0
Original line number Original line Diff line number Diff line
@@ -67,6 +67,7 @@ import android.util.Log;
import android.util.Pair;
import android.util.Pair;
import android.util.SparseSetArray;
import android.util.SparseSetArray;
import android.view.Display;
import android.view.Display;
import android.view.View;
import android.view.ViewGroup;
import android.view.ViewGroup;
import android.view.WindowManager;
import android.view.WindowManager;


@@ -394,6 +395,7 @@ public class BubbleController implements ConfigurationController.ConfigurationLi
                : statusBarService;
                : statusBarService;


        mBubbleScrim = new ScrimView(mContext);
        mBubbleScrim = new ScrimView(mContext);
        mBubbleScrim.setImportantForAccessibility(View.IMPORTANT_FOR_ACCESSIBILITY_NO);


        mSavedBubbleKeysPerUser = new SparseSetArray<>();
        mSavedBubbleKeysPerUser = new SparseSetArray<>();
        mCurrentUserId = mNotifUserManager.getCurrentUserId();
        mCurrentUserId = mNotifUserManager.getCurrentUserId();
+4 −0
Original line number Original line Diff line number Diff line
@@ -2470,6 +2470,10 @@ public class BubbleStackView extends FrameLayout
                    .spring(DynamicAnimation.SCALE_Y, 1f)
                    .spring(DynamicAnimation.SCALE_Y, 1f)
                    .spring(DynamicAnimation.TRANSLATION_X, targetX)
                    .spring(DynamicAnimation.TRANSLATION_X, targetX)
                    .spring(DynamicAnimation.TRANSLATION_Y, targetY)
                    .spring(DynamicAnimation.TRANSLATION_Y, targetY)
                    .withEndActions(() -> {
                        View child = mManageMenu.getChildAt(0);
                        child.requestAccessibilityFocus();
                    })
                    .start();
                    .start();


            mManageMenu.setVisibility(View.VISIBLE);
            mManageMenu.setVisibility(View.VISIBLE);