From 52ff3258410fc66a389e3448a487625f4a895117 Mon Sep 17 00:00:00 2001 From: Jon Miranda Date: Fri, 14 Jul 2017 11:39:16 -0700 Subject: [PATCH] Move padding from Hotseat to Hotseat's CellLayout. Bug: 63590728 Change-Id: I3ba6ba83b83897d75cc4445a7dab9c5b8b7b8ff1 --- src/com/android/launcher3/DeviceProfile.java | 16 ++++++++++------ 1 file changed, 10 insertions(+), 6 deletions(-) diff --git a/src/com/android/launcher3/DeviceProfile.java b/src/com/android/launcher3/DeviceProfile.java index 0f4abbe2e9..6b76cd62f7 100644 --- a/src/com/android/launcher3/DeviceProfile.java +++ b/src/com/android/launcher3/DeviceProfile.java @@ -597,15 +597,18 @@ public class DeviceProfile { ? hotseatBarLeftNavBarRightPaddingPx : hotseatBarRightNavBarRightPaddingPx; - hotseat.getLayout().setPadding(mInsets.left, mInsets.top, mInsets.right + paddingRight, + hotseat.getLayout().setPadding(mInsets.left + cellLayoutPaddingLeftRightPx, + mInsets.top, mInsets.right + paddingRight + cellLayoutPaddingLeftRightPx, workspacePadding.bottom); } else if (isTablet) { // Pad the hotseat with the workspace padding calculated above lp.gravity = Gravity.BOTTOM; lp.width = LayoutParams.MATCH_PARENT; lp.height = hotseatBarHeightPx + mInsets.bottom; - hotseat.getLayout().setPadding(hotseatAdjustment + workspacePadding.left, - hotseatBarTopPaddingPx, hotseatAdjustment + workspacePadding.right, + hotseat.getLayout().setPadding(hotseatAdjustment + workspacePadding.left + + cellLayoutPaddingLeftRightPx, + hotseatBarTopPaddingPx, + hotseatAdjustment + workspacePadding.right + cellLayoutPaddingLeftRightPx, hotseatBarBottomPaddingPx + mInsets.bottom); } else { // For phones, layout the hotseat without any bottom margin @@ -613,11 +616,12 @@ public class DeviceProfile { lp.gravity = Gravity.BOTTOM; lp.width = LayoutParams.MATCH_PARENT; lp.height = hotseatBarHeightPx + mInsets.bottom; - hotseat.getLayout().setPadding(hotseatAdjustment + workspacePadding.left, - hotseatBarTopPaddingPx, hotseatAdjustment + workspacePadding.right, + hotseat.getLayout().setPadding(hotseatAdjustment + workspacePadding.left + + cellLayoutPaddingLeftRightPx, + hotseatBarTopPaddingPx, + hotseatAdjustment + workspacePadding.right + cellLayoutPaddingLeftRightPx, hotseatBarBottomPaddingPx + mInsets.bottom); } - hotseat.setPadding(cellLayoutPaddingLeftRightPx, 0, cellLayoutPaddingLeftRightPx, 0); hotseat.setLayoutParams(lp); // Layout the page indicators