summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
author Riddle Hsu <riddlehsu@google.com> 2024-11-18 04:20:06 +0000
committer Riddle Hsu <riddlehsu@google.com> 2024-11-18 04:37:28 +0000
commit4c6a5d37622f971a4fc12c943bd614cbf6d87bdc (patch)
tree5d8fb4a96121454fc7af77dc39e14eff4122ca47
parent4ec85a58d5a712b1ef89153998b491cfc6d0626a (diff)
Set background color of more actions pill
This aligns the same logic as bindAppInfoPill and bindWindowingPill. The background color should be set on the container rather than the menu items. Bug: 379176148 Flag: EXEMPT bugfix Test: Switch between app with dark and light background. Press app handle on top. The background color of menu item should be updated. Change-Id: I73854817b8d5fca7c65418ce5ceba693eacda5e1
-rw-r--r--libs/WindowManager/Shell/src/com/android/wm/shell/windowdecor/HandleMenu.kt3
1 files changed, 2 insertions, 1 deletions
diff --git a/libs/WindowManager/Shell/src/com/android/wm/shell/windowdecor/HandleMenu.kt b/libs/WindowManager/Shell/src/com/android/wm/shell/windowdecor/HandleMenu.kt
index 54c247bff984..c98c6a39e305 100644
--- a/libs/WindowManager/Shell/src/com/android/wm/shell/windowdecor/HandleMenu.kt
+++ b/libs/WindowManager/Shell/src/com/android/wm/shell/windowdecor/HandleMenu.kt
@@ -650,6 +650,8 @@ class HandleMenu(
}
private fun bindMoreActionsPill(style: MenuStyle) {
+ moreActionsPill.background.setTint(style.backgroundColor)
+
arrayOf(
screenshotBtn to SHOULD_SHOW_SCREENSHOT_BUTTON,
newWindowBtn to shouldShowNewWindowButton,
@@ -660,7 +662,6 @@ class HandleMenu(
val shouldShow = it.second
button.apply {
isGone = !shouldShow
- background.setTint(style.backgroundColor)
setTextColor(style.textColor)
compoundDrawableTintList = ColorStateList.valueOf(style.textColor)
}