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

Commit be9798b6 authored by Winson's avatar Winson
Browse files

Fixing issue with shadow drawing over search bar.

- Adding notion of clip-against view for click shadow alignment.

Bug: 30255227
Change-Id: Id5716a3484051a55690025d61f709e3d96cbe024
parent 2eeae10e
Loading
Loading
Loading
Loading
+2 −1
Original line number Diff line number Diff line
@@ -406,7 +406,8 @@ public class CellLayout extends ViewGroup implements BubbleTextShadowHandler {
            mTouchFeedbackView.animate().cancel();
        } else {
            if (mTouchFeedbackView.setBitmap(background)) {
                mTouchFeedbackView.alignWithIconView(icon, mShortcutsAndWidgets);
                mTouchFeedbackView.alignWithIconView(icon, mShortcutsAndWidgets,
                        null /* clipAgainstView */);
                mTouchFeedbackView.animateShadow();
            }
        }
+16 −1
Original line number Diff line number Diff line
@@ -21,6 +21,7 @@ import android.graphics.Bitmap;
import android.graphics.Canvas;
import android.graphics.Color;
import android.graphics.Paint;
import android.graphics.Rect;
import android.view.View;
import android.view.ViewDebug;
import android.view.ViewGroup;
@@ -91,13 +92,27 @@ public class ClickShadowView extends View {
     * Aligns the shadow with {@param view}
     * @param viewParent immediate parent of {@param view}. It must be a sibling of this view.
     */
    public void alignWithIconView(BubbleTextView view, ViewGroup viewParent) {
    public void alignWithIconView(BubbleTextView view, ViewGroup viewParent, View clipAgainstView) {
        float leftShift = view.getLeft() + viewParent.getLeft() - getLeft();
        float topShift = view.getTop() + viewParent.getTop() - getTop();
        int iconWidth = view.getRight() - view.getLeft();
        int iconHeight = view.getBottom() - view.getTop();
        int iconHSpace = iconWidth - view.getCompoundPaddingRight() - view.getCompoundPaddingLeft();
        float drawableWidth = view.getIcon().getBounds().width();

        if (clipAgainstView != null) {
            // Set the bounds to clip against
            int[] coords = new int[] {0, 0};
            Utilities.getDescendantCoordRelativeToAncestor(clipAgainstView, (View) getParent(),
                    coords, false);
            int clipLeft = (int) Math.max(0, coords[0] - leftShift - mShadowPadding);
            int clipTop = (int) Math.max(0, coords[1] - topShift - mShadowPadding) ;
            setClipBounds(new Rect(clipLeft, clipTop, clipLeft + iconWidth, clipTop + iconHeight));
        } else {
            // Reset the clip bounds
            setClipBounds(null);
        }

        setTranslationX(leftShift
                + viewParent.getTranslationX()
                + view.getCompoundPaddingLeft() * view.getScaleX()
+2 −2
Original line number Diff line number Diff line
@@ -113,8 +113,8 @@ public final class Utilities {
                && "NMR1".compareTo(VERSION.CODENAME) <= 0;
    }

    // TODO: use Build.VERSION_CODES when available
    public static final boolean ATLEAST_MARSHMALLOW = Build.VERSION.SDK_INT >= 23;
    public static final boolean ATLEAST_MARSHMALLOW =
            Build.VERSION.SDK_INT >= Build.VERSION_CODES.M;

    public static final boolean ATLEAST_LOLLIPOP_MR1 =
            Build.VERSION.SDK_INT >= Build.VERSION_CODES.LOLLIPOP_MR1;
+4 −1
Original line number Diff line number Diff line
@@ -18,6 +18,7 @@ package com.android.launcher3.allapps;
import android.content.Context;
import android.graphics.Bitmap;
import android.util.AttributeSet;
import android.view.View;
import android.view.ViewGroup;
import android.widget.FrameLayout;

@@ -26,6 +27,7 @@ import com.android.launcher3.BubbleTextView.BubbleTextShadowHandler;
import com.android.launcher3.ClickShadowView;
import com.android.launcher3.DeviceProfile;
import com.android.launcher3.Launcher;
import com.android.launcher3.R;

/**
 * A container for RecyclerView to allow for the click shadow view to be shown behind an icon that
@@ -63,7 +65,8 @@ public class AllAppsRecyclerViewContainerView extends FrameLayout
            mTouchFeedbackView.setBitmap(null);
            mTouchFeedbackView.animate().cancel();
        } else if (mTouchFeedbackView.setBitmap(background)) {
            mTouchFeedbackView.alignWithIconView(icon, (ViewGroup) icon.getParent());
            View rv = findViewById(R.id.apps_list_view);
            mTouchFeedbackView.alignWithIconView(icon, (ViewGroup) icon.getParent(), rv);
            mTouchFeedbackView.animateShadow();
        }
    }