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

Commit c31ddde1 authored by Mady Mellor's avatar Mady Mellor
Browse files

Add alpha back to bubble scrim

Test: atest ScrimControllerTest
Test: manual - expand bubble stack, observe scrim
Fixes: 181791886
Change-Id: I254856cd9f0775bb52cb47197d78e40d78853b38
parent 5efdaa63
Loading
Loading
Loading
Loading
+7 −2
Original line number Diff line number Diff line
@@ -123,6 +123,12 @@ public class ScrimController implements ViewTreeObserver.OnPreDrawListener, Dump
     */
    public static final float BUSY_SCRIM_ALPHA = 1f;

    /**
     * The default scrim under the expanded bubble stack.
     * This should not be lower than 0.54, otherwise we won't pass GAR.
     */
    public static final float BUBBLE_SCRIM_ALPHA = 0.6f;

    /**
     * Scrim opacity that can have text on top.
     */
@@ -207,8 +213,7 @@ public class ScrimController implements ViewTreeObserver.OnPreDrawListener, Dump
            FeatureFlags featureFlags, @Main Executor mainExecutor) {
        mScrimStateListener = lightBarController::setScrimState;
        mDefaultScrimAlpha = featureFlags.isShadeOpaque() ? BUSY_SCRIM_ALPHA : GAR_SCRIM_ALPHA;
        ScrimState.BUBBLE_EXPANDED.setBubbleAlpha(featureFlags.isShadeOpaque()
                ? BUSY_SCRIM_ALPHA : GAR_SCRIM_ALPHA);
        ScrimState.BUBBLE_EXPANDED.setBubbleAlpha(BUBBLE_SCRIM_ALPHA);
        mBlurUtils = blurUtils;

        mKeyguardStateController = keyguardStateController;
+1 −1
Original line number Diff line number Diff line
@@ -542,7 +542,7 @@ public class ScrimControllerTest extends SysuiTestCase {
        Assert.assertEquals(ScrimController.BUSY_SCRIM_ALPHA,
                mScrimBehind.getViewAlpha(), 0.0f);
        // Bubble scrim should be visible
        Assert.assertEquals(ScrimController.BUSY_SCRIM_ALPHA,
        Assert.assertEquals(ScrimController.BUBBLE_SCRIM_ALPHA,
                mScrimForBubble.getViewAlpha(), 0.0f);
    }