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

Commit 8d39bdab authored by Wenbo Jie's avatar Wenbo Jie
Browse files

[DocsUI M3] Fixed the move issue for filter chips 

After moving the filter chips padding from the container to the
chip group, we need to update the code to use padding start as
it's moving origin.

Check the fixed demo in the bug.

Bug: 384574562
Test: Manual inpsection, click/tap the filter chips
Flag: com.android.documentsui.flags.use_material3
Change-Id: I34151e53f5bd4ac10dfdec3d4567b50cb5167cf1
parent 07f9eded

src/com/android/documentsui/queries/SearchChipViewManager.java

+3 −3
Original line number Diff line number Diff line
@@ -454,13 +454,13 @@ public class SearchChipViewManager { 
                                .getResources()

                                .getDimensionPixelSize(R.dimen.search_chip_spacing);

        final boolean isRtl = mChipGroup.getLayoutDirection() == View.LAYOUT_DIRECTION_RTL;

        final float chipMarginStartEnd =

        final float chipGroupPaddingStart =

                useMaterial3()

                        ? 0

                        ? mChipGroup.getPaddingStart()

                        : mChipGroup

                                .getResources()

                                .getDimensionPixelSize(R.dimen.search_chip_half_spacing);

        float lastX = isRtl ? mChipGroup.getWidth() - chipMarginStartEnd : chipMarginStartEnd;

        float lastX = isRtl ? mChipGroup.getWidth() - chipGroupPaddingStart : chipGroupPaddingStart;



        // remove all chips except current clicked chip to avoid losing

        // accessibility focus.