From 25b2ac85277d948b866ebafb0f0483d64799f2fc Mon Sep 17 00:00:00 2001 From: Brian Isganitis Date: Fri, 28 Jan 2022 16:25:13 -0500 Subject: [PATCH] Do not scale down taskbar all apps icons. Test: Manual Bug: 204696617 Change-Id: I2ede15d8beaef65661984fe63bd99d0e7e726774 --- .../taskbar/TaskbarActivityContext.java | 1 + src/com/android/launcher3/DeviceProfile.java | 48 +++++++++++-------- 2 files changed, 30 insertions(+), 19 deletions(-) diff --git a/quickstep/src/com/android/launcher3/taskbar/TaskbarActivityContext.java b/quickstep/src/com/android/launcher3/taskbar/TaskbarActivityContext.java index e99edb5d13..abe8aad277 100644 --- a/quickstep/src/com/android/launcher3/taskbar/TaskbarActivityContext.java +++ b/quickstep/src/com/android/launcher3/taskbar/TaskbarActivityContext.java @@ -234,6 +234,7 @@ public class TaskbarActivityContext extends ContextThemeWrapper implements Activ mDeviceProfile.updateIconSize(1, resources); float iconScale = taskbarIconSize / mDeviceProfile.iconSizePx; mDeviceProfile.updateIconSize(iconScale, resources); + mDeviceProfile.updateAllAppsIconSize(1, resources); // Leave all apps unscaled. } /** Creates LayoutParams for adding a view directly to WindowManager as a new window */ diff --git a/src/com/android/launcher3/DeviceProfile.java b/src/com/android/launcher3/DeviceProfile.java index d3486027dd..295e84e1b8 100644 --- a/src/com/android/launcher3/DeviceProfile.java +++ b/src/com/android/launcher3/DeviceProfile.java @@ -726,25 +726,7 @@ public class DeviceProfile { } // All apps - if (numShownAllAppsColumns != inv.numColumns) { - allAppsIconSizePx = - pxFromDp(inv.allAppsIconSize[mTypeIndex], mMetrics); - allAppsIconTextSizePx = - pxFromSp(inv.allAppsIconTextSize[mTypeIndex], mMetrics); - allAppsIconDrawablePaddingPx = iconDrawablePaddingOriginalPx; - autoResizeAllAppsCells(); - } else { - allAppsIconSizePx = iconSizePx; - allAppsIconTextSizePx = iconTextSizePx; - allAppsIconDrawablePaddingPx = iconDrawablePaddingPx; - allAppsCellHeightPx = getCellSize().y; - } - allAppsCellWidthPx = allAppsIconSizePx + (2 * allAppsIconDrawablePaddingPx); - updateAllAppsWidth(res); - - if (isVerticalLayout) { - hideWorkspaceLabelsIfNotEnoughSpace(); - } + updateAllAppsIconSize(scale, res); // Hotseat hotseatBorderSpace = pxFromDp(inv.hotseatBorderSpaces[mTypeIndex], mMetrics, scale); @@ -772,6 +754,34 @@ public class DeviceProfile { folderIconOffsetYPx = (iconSizePx - folderIconSizePx) / 2; } + + /** + * Updates the iconSize for allApps* variants. + */ + public void updateAllAppsIconSize(float scale, Resources res) { + if (numShownAllAppsColumns != inv.numColumns) { + allAppsIconSizePx = + pxFromDp(inv.allAppsIconSize[mTypeIndex], mMetrics); + allAppsIconTextSizePx = + pxFromSp(inv.allAppsIconTextSize[mTypeIndex], mMetrics); + allAppsIconDrawablePaddingPx = iconDrawablePaddingOriginalPx; + autoResizeAllAppsCells(); + } else { + float invIconSizeDp = inv.iconSize[mTypeIndex]; + float invIconTextSizeSp = inv.iconTextSize[mTypeIndex]; + allAppsIconSizePx = Math.max(1, pxFromDp(invIconSizeDp, mMetrics, scale)); + allAppsIconTextSizePx = (int) (pxFromSp(invIconTextSizeSp, mMetrics) * scale); + allAppsIconDrawablePaddingPx = (int) (iconDrawablePaddingOriginalPx * scale); + allAppsCellHeightPx = getCellSize().y; + } + + allAppsCellWidthPx = allAppsIconSizePx + (2 * allAppsIconDrawablePaddingPx); + updateAllAppsWidth(res); + if (isVerticalBarLayout()) { + hideWorkspaceLabelsIfNotEnoughSpace(); + } + } + private void updateAvailableFolderCellDimensions(Resources res) { updateFolderCellSize(1f, res);