Merge pull request #123 from esensar/fix/popup-menu-you-colors
Fix popup menu colors when MaterialColors returns -1
This commit is contained in:
commit
a1a960428a
|
@ -25,6 +25,7 @@ import com.simplemobiletools.commons.extensions.getPopupMenuTheme
|
||||||
import com.simplemobiletools.commons.extensions.getProperTextColor
|
import com.simplemobiletools.commons.extensions.getProperTextColor
|
||||||
import com.simplemobiletools.commons.extensions.showErrorToast
|
import com.simplemobiletools.commons.extensions.showErrorToast
|
||||||
import com.simplemobiletools.commons.helpers.isQPlus
|
import com.simplemobiletools.commons.helpers.isQPlus
|
||||||
|
import com.simplemobiletools.commons.helpers.isSPlus
|
||||||
import com.simplemobiletools.launcher.R
|
import com.simplemobiletools.launcher.R
|
||||||
import com.simplemobiletools.launcher.activities.SettingsActivity
|
import com.simplemobiletools.launcher.activities.SettingsActivity
|
||||||
import com.simplemobiletools.launcher.helpers.ITEM_TYPE_FOLDER
|
import com.simplemobiletools.launcher.helpers.ITEM_TYPE_FOLDER
|
||||||
|
@ -90,7 +91,12 @@ fun Activity.handleGridItemPopupMenu(anchorView: View, gridItem: HomeScreenGridI
|
||||||
|
|
||||||
inflate(R.menu.menu_app_icon)
|
inflate(R.menu.menu_app_icon)
|
||||||
menu.forEach {
|
menu.forEach {
|
||||||
val color = MaterialColors.getColor(contextTheme, android.R.attr.textColorPrimary, getProperTextColor())
|
val default = getProperTextColor()
|
||||||
|
val color = if (isSPlus() && config.isUsingSystemTheme) {
|
||||||
|
default
|
||||||
|
} else {
|
||||||
|
MaterialColors.getColor(contextTheme, android.R.attr.actionMenuTextColor, default)
|
||||||
|
}
|
||||||
it.iconTintList = ColorStateList.valueOf(color)
|
it.iconTintList = ColorStateList.valueOf(color)
|
||||||
}
|
}
|
||||||
menu.findItem(R.id.rename).isVisible = (gridItem.type == ITEM_TYPE_ICON || gridItem.type == ITEM_TYPE_FOLDER) && !isOnAllAppsFragment
|
menu.findItem(R.id.rename).isVisible = (gridItem.type == ITEM_TYPE_ICON || gridItem.type == ITEM_TYPE_FOLDER) && !isOnAllAppsFragment
|
||||||
|
|
Loading…
Reference in New Issue