diff options
-rw-r--r-- | res/color-night-v31/popup_color_background.xml | 20 | ||||
-rw-r--r-- | res/color-v31/popup_color_background.xml | 20 | ||||
-rw-r--r-- | res/color/popup_color_background.xml | 18 | ||||
-rw-r--r-- | res/drawable/popup_background.xml | 2 | ||||
-rw-r--r-- | res/values/attrs.xml | 1 | ||||
-rw-r--r-- | res/values/styles.xml | 2 | ||||
-rw-r--r-- | src/com/android/launcher3/popup/ArrowPopup.java | 28 | ||||
-rw-r--r-- | src/com/android/launcher3/views/OptionsPopupView.java | 5 |
8 files changed, 15 insertions, 81 deletions
diff --git a/res/color-night-v31/popup_color_background.xml b/res/color-night-v31/popup_color_background.xml deleted file mode 100644 index 13ceaa09e7..0000000000 --- a/res/color-night-v31/popup_color_background.xml +++ /dev/null @@ -1,20 +0,0 @@ -<?xml version="1.0" encoding="utf-8"?> -<!-- Copyright (C) 2023 The Android Open Source Project - - Licensed under the Apache License, Version 2.0 (the "License"); - you may not use this file except in compliance with the License. - You may obtain a copy of the License at - - http://www.apache.org/licenses/LICENSE-2.0 - - Unless required by applicable law or agreed to in writing, software - distributed under the License is distributed on an "AS IS" BASIS, - WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. - See the License for the specific language governing permissions and - limitations under the License. ---> -<selector xmlns:android="http://schemas.android.com/apk/res/android" > - <item - android:color="@android:color/system_neutral1_900" - android:lStar="12" /> -</selector> diff --git a/res/color-v31/popup_color_background.xml b/res/color-v31/popup_color_background.xml deleted file mode 100644 index 99155d8972..0000000000 --- a/res/color-v31/popup_color_background.xml +++ /dev/null @@ -1,20 +0,0 @@ -<?xml version="1.0" encoding="utf-8"?> -<!-- Copyright (C) 2023 The Android Open Source Project - - Licensed under the Apache License, Version 2.0 (the "License"); - you may not use this file except in compliance with the License. - You may obtain a copy of the License at - - http://www.apache.org/licenses/LICENSE-2.0 - - Unless required by applicable law or agreed to in writing, software - distributed under the License is distributed on an "AS IS" BASIS, - WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. - See the License for the specific language governing permissions and - limitations under the License. ---> -<selector xmlns:android="http://schemas.android.com/apk/res/android" > - <item - android:color="@android:color/system_neutral1_50" - android:lStar="94" /> -</selector> diff --git a/res/color/popup_color_background.xml b/res/color/popup_color_background.xml deleted file mode 100644 index e87e77231c..0000000000 --- a/res/color/popup_color_background.xml +++ /dev/null @@ -1,18 +0,0 @@ -<?xml version="1.0" encoding="utf-8"?> -<!-- Copyright (C) 2023 The Android Open Source Project - - Licensed under the Apache License, Version 2.0 (the "License"); - you may not use this file except in compliance with the License. - You may obtain a copy of the License at - - http://www.apache.org/licenses/LICENSE-2.0 - - Unless required by applicable law or agreed to in writing, software - distributed under the License is distributed on an "AS IS" BASIS, - WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. - See the License for the specific language governing permissions and - limitations under the License. ---> -<selector xmlns:android="http://schemas.android.com/apk/res/android" > - <item android:color="?attr/popupColorBackground" /> -</selector> diff --git a/res/drawable/popup_background.xml b/res/drawable/popup_background.xml index 6eedecb6ca..4ddd228c68 100644 --- a/res/drawable/popup_background.xml +++ b/res/drawable/popup_background.xml @@ -15,6 +15,6 @@ --> <shape xmlns:android="http://schemas.android.com/apk/res/android" android:shape="rectangle"> - <solid android:color="?attr/popupColorBackground"/> + <solid android:color="?attr/materialColorSurfaceContainer"/> <corners android:radius="@dimen/dialogCornerRadius"/> </shape>
\ No newline at end of file diff --git a/res/values/attrs.xml b/res/values/attrs.xml index 6151b5fb93..57c9bc7a8f 100644 --- a/res/values/attrs.xml +++ b/res/values/attrs.xml @@ -25,7 +25,6 @@ <attr name="popupColorPrimary" format="color" /> <attr name="popupColorSecondary" format="color" /> <attr name="popupColorTertiary" format="color" /> - <attr name="popupColorBackground" format="color" /> <attr name="popupTextColor" format="color" /> <attr name="popupShadeFirst" format="color" /> <attr name="popupShadeSecond" format="color" /> diff --git a/res/values/styles.xml b/res/values/styles.xml index ee7ed2687f..4a92e732c7 100644 --- a/res/values/styles.xml +++ b/res/values/styles.xml @@ -86,7 +86,6 @@ <item name="popupColorPrimary">@color/popup_color_primary_light</item> <item name="popupColorSecondary">@color/popup_color_secondary_light</item> <item name="popupColorTertiary">@color/popup_color_tertiary_light</item> - <item name="popupColorBackground">#EFEDED</item> <item name="popupTextColor">@color/system_on_surface_light</item> <item name="popupShadeFirst">@color/popup_shade_first_light</item> <item name="popupShadeSecond">@color/popup_shade_second_light</item> @@ -161,7 +160,6 @@ <item name="popupColorPrimary">@color/popup_color_primary_dark</item> <item name="popupColorSecondary">@color/popup_color_secondary_dark</item> <item name="popupColorTertiary">@color/popup_color_tertiary_dark</item> - <item name="popupColorBackground">#1F2020</item> <item name="popupTextColor">@color/system_on_surface_dark</item> <item name="popupNotificationDotColor">@color/popup_notification_dot_dark</item> <item name="popupShadeFirst">@color/popup_shade_first_dark</item> diff --git a/src/com/android/launcher3/popup/ArrowPopup.java b/src/com/android/launcher3/popup/ArrowPopup.java index 4d4a8f749a..c2debfa2ed 100644 --- a/src/com/android/launcher3/popup/ArrowPopup.java +++ b/src/com/android/launcher3/popup/ArrowPopup.java @@ -16,8 +16,6 @@ package com.android.launcher3.popup; -import static androidx.core.content.ContextCompat.getColorStateList; - import static com.android.app.animation.Interpolators.EMPHASIZED_ACCELERATE; import static com.android.app.animation.Interpolators.EMPHASIZED_DECELERATE; import static com.android.app.animation.Interpolators.LINEAR; @@ -56,8 +54,6 @@ import com.android.launcher3.util.Themes; import com.android.launcher3.views.ActivityContext; import com.android.launcher3.views.BaseDragLayer; -import java.util.Arrays; - /** * A container for shortcuts to deep links and notifications associated with an app. * @@ -130,7 +126,7 @@ public abstract class ArrowPopup<T extends Context & ActivityContext> // Tag for Views that have children that will need to be iterated to add styling. private final String mIterateChildrenTag; - protected final int[] mColorIds; + protected final int[] mColors; public ArrowPopup(Context context, AttributeSet attrs, int defStyleAttr) { super(context, attrs, defStyleAttr); @@ -142,8 +138,7 @@ public abstract class ArrowPopup<T extends Context & ActivityContext> // Initialize arrow view final Resources resources = getResources(); - mArrowColor = getColorStateList(getContext(), R.color.popup_color_background) - .getDefaultColor(); + mArrowColor = Themes.getAttrColor(getContext(), R.attr.materialColorSurfaceContainer); mChildContainerMargin = resources.getDimensionPixelSize(R.dimen.popup_margin); mArrowWidth = resources.getDimensionPixelSize(R.dimen.popup_arrow_width); mArrowHeight = resources.getDimensionPixelSize(R.dimen.popup_arrow_height); @@ -158,21 +153,25 @@ public abstract class ArrowPopup<T extends Context & ActivityContext> mRoundedTop = new GradientDrawable(); int popupPrimaryColor = Themes.getAttrColor(context, R.attr.popupColorPrimary); mRoundedTop.setColor(popupPrimaryColor); - mRoundedTop.setCornerRadii(new float[] { mOutlineRadius, mOutlineRadius, mOutlineRadius, + mRoundedTop.setCornerRadii(new float[]{mOutlineRadius, mOutlineRadius, mOutlineRadius, mOutlineRadius, smallerRadius, smallerRadius, smallerRadius, smallerRadius}); mRoundedBottom = new GradientDrawable(); mRoundedBottom.setColor(popupPrimaryColor); - mRoundedBottom.setCornerRadii(new float[] { smallerRadius, smallerRadius, smallerRadius, + mRoundedBottom.setCornerRadii(new float[]{smallerRadius, smallerRadius, smallerRadius, smallerRadius, mOutlineRadius, mOutlineRadius, mOutlineRadius, mOutlineRadius}); mIterateChildrenTag = getContext().getString(R.string.popup_container_iterate_children); if (mActivityContext.canUseMultipleShadesForPopup()) { - mColorIds = new int[]{R.color.popup_shade_first, R.color.popup_shade_second, - R.color.popup_shade_third}; + mColors = new int[]{ + getContext().getColor(R.color.popup_shade_first), + getContext().getColor(R.color.popup_shade_second), + getContext().getColor(R.color.popup_shade_third) + }; } else { - mColorIds = new int[]{R.color.popup_color_background}; + mColors = new int[]{Themes.getAttrColor(getContext(), + R.attr.materialColorSurfaceContainer)}; } } @@ -219,15 +218,14 @@ public abstract class ArrowPopup<T extends Context & ActivityContext> } /** - * @param backgroundColor When Color.TRANSPARENT, we get color from {@link #mColorIds}. + * @param backgroundColor When Color.TRANSPARENT, we get color from {@link #mColors}. * Otherwise, we will use this color for all child views. */ protected void assignMarginsAndBackgrounds(ViewGroup viewGroup, int backgroundColor) { int[] colors = null; if (backgroundColor == Color.TRANSPARENT) { // Lazily get the colors so they match the current wallpaper colors. - colors = Arrays.stream(mColorIds).map( - r -> getColorStateList(getContext(), r).getDefaultColor()).toArray(); + colors = mColors; } int count = viewGroup.getChildCount(); diff --git a/src/com/android/launcher3/views/OptionsPopupView.java b/src/com/android/launcher3/views/OptionsPopupView.java index 62eed5c545..82cc40d93b 100644 --- a/src/com/android/launcher3/views/OptionsPopupView.java +++ b/src/com/android/launcher3/views/OptionsPopupView.java @@ -15,8 +15,6 @@ */ package com.android.launcher3.views; -import static androidx.core.content.ContextCompat.getColorStateList; - import static com.android.launcher3.BuildConfig.WIDGETS_ENABLED; import static com.android.launcher3.LauncherState.EDIT_MODE; import static com.android.launcher3.config.FeatureFlags.MULTI_SELECT_EDIT_MODE; @@ -147,8 +145,7 @@ public class OptionsPopupView<T extends Context & ActivityContext> extends Arrow @Override public void assignMarginsAndBackgrounds(ViewGroup viewGroup) { - assignMarginsAndBackgrounds(viewGroup, - getColorStateList(getContext(), mColorIds[0]).getDefaultColor()); + assignMarginsAndBackgrounds(viewGroup, mColors[0]); // last shortcut doesn't need bottom margin final int count = viewGroup.getChildCount() - 1; for (int i = 0; i < count; i++) { |