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

Commit 478bb131 authored by Matt Casey's avatar Matt Casey Committed by Automerger Merge Worker
Browse files

Merge "Minimize distance between text and pin in direct share row" into tm-dev... 

Merge "Minimize distance between text and pin in direct share row" into tm-dev am: 59ebb04c am: c3e50ddb am: cb002db9

Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/base/+/18521525



Change-Id: I87d2b95e1b36c18083c723c365c56b1f99501fc2
Signed-off-by: default avatarAutomerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
parents 25245a45 cb002db9

core/java/com/android/internal/app/ChooserListAdapter.java

+38 −0
Original line number Diff line number Diff line
@@ -36,9 +36,11 @@ import android.os.UserHandle; 
import android.os.UserManager;

import android.provider.DeviceConfig;

import android.service.chooser.ChooserTarget;

import android.text.Layout;

import android.util.Log;

import android.view.View;

import android.view.ViewGroup;

import android.widget.TextView;



import com.android.internal.R;

import com.android.internal.app.ResolverActivity.ResolvedComponentInfo;
@@ -102,6 +104,37 @@ public class ChooserListAdapter extends ResolverListAdapter { 
    private AppPredictor mAppPredictor;

    private AppPredictor.Callback mAppPredictorCallback;



    // For pinned direct share labels, if the text spans multiple lines, the TextView will consume

    // the full width, even if the characters actually take up less than that. Measure the actual

    // line widths and constrain the View's width based upon that so that the pin doesn't end up

    // very far from the text.

    private final View.OnLayoutChangeListener mPinTextSpacingListener =

            new View.OnLayoutChangeListener() {

                @Override

                public void onLayoutChange(View v, int left, int top, int right, int bottom,

                        int oldLeft, int oldTop, int oldRight, int oldBottom) {

                    TextView textView = (TextView) v;

                    Layout layout = textView.getLayout();

                    if (layout != null) {

                        int textWidth = 0;

                        for (int line = 0; line < layout.getLineCount(); line++) {

                            textWidth = Math.max((int) Math.ceil(layout.getLineMax(line)),

                                    textWidth);

                        }

                        int desiredWidth = textWidth + textView.getPaddingLeft()

                                + textView.getPaddingRight();

                        if (textView.getWidth() > desiredWidth) {

                            ViewGroup.LayoutParams params = textView.getLayoutParams();

                            params.width = desiredWidth;

                            textView.setLayoutParams(params);

                            // Need to wait until layout pass is over before requesting layout.

                            textView.post(() -> textView.requestLayout());

                        }

                        textView.removeOnLayoutChangeListener(this);

                    }

                }

            };



    public ChooserListAdapter(Context context, List<Intent> payloadIntents,

            Intent[] initialIntents, List<ResolveInfo> rList,

            boolean filterLastUsed, ResolverListController resolverListController,
@@ -225,6 +258,7 @@ public class ChooserListAdapter extends ResolverListAdapter { 
    @Override

    protected void onBindView(View view, TargetInfo info, int position) {

        final ViewHolder holder = (ViewHolder) view.getTag();



        if (info == null) {

            holder.icon.setImageDrawable(

                    mContext.getDrawable(R.drawable.resolver_icon_placeholder));
@@ -274,6 +308,9 @@ public class ChooserListAdapter extends ResolverListAdapter { 
            holder.itemView.setBackground(holder.defaultItemViewBackground);

        }



        // Always remove the spacing listener, attach as needed to direct share targets below.

        holder.text.removeOnLayoutChangeListener(mPinTextSpacingListener);



        if (info instanceof MultiDisplayResolveInfo) {

            // If the target is grouped show an indicator

            Drawable bkg = mContext.getDrawable(R.drawable.chooser_group_background);
@@ -286,6 +323,7 @@ public class ChooserListAdapter extends ResolverListAdapter { 
            Drawable bkg = mContext.getDrawable(R.drawable.chooser_pinned_background);

            holder.text.setPaddingRelative(bkg.getIntrinsicWidth() /* start */, 0, 0, 0);

            holder.text.setBackground(bkg);

            holder.text.addOnLayoutChangeListener(mPinTextSpacingListener);

        } else {

            holder.text.setBackground(null);

            holder.text.setPaddingRelative(0, 0, 0, 0);