From ba57dba13b6cca616af0e29d5c77781c43b734d8 Mon Sep 17 00:00:00 2001
From: Matt Pietal <mpietal@google.com>
Date: Thu, 27 May 2021 15:22:24 -0400
Subject: [PATCH] Smartspace padding for AOD/LS

Update spacing, specifically to match the end padding for the
smartspace container. Reduce padding for the icon shelf to match
incoming changes for smartspace

Bug: 189363380
Test: manual
Change-Id: I82030e21cd5b77424b30dc3c634d01c73beac148
---
 .../SystemUI/res-keyguard/layout/keyguard_clock_switch.xml  | 2 +-
 packages/SystemUI/res-keyguard/values/dimens.xml            | 3 ++-
 .../com/android/keyguard/KeyguardClockSwitchController.java | 6 ++++--
 3 files changed, 7 insertions(+), 4 deletions(-)

diff --git a/packages/SystemUI/res-keyguard/layout/keyguard_clock_switch.xml b/packages/SystemUI/res-keyguard/layout/keyguard_clock_switch.xml
index 35eebac475373..9811434906b78 100644
--- a/packages/SystemUI/res-keyguard/layout/keyguard_clock_switch.xml
+++ b/packages/SystemUI/res-keyguard/layout/keyguard_clock_switch.xml
@@ -80,7 +80,7 @@
         android:layout_width="match_parent"
         android:layout_height="@dimen/notification_shelf_height"
         android:layout_below="@id/keyguard_status_area"
-        android:paddingStart="@dimen/below_clock_padding_start_extra"
+        android:paddingStart="@dimen/below_clock_padding_start_icons"
         android:visibility="invisible"
     />
 </com.android.keyguard.KeyguardClockSwitch>
diff --git a/packages/SystemUI/res-keyguard/values/dimens.xml b/packages/SystemUI/res-keyguard/values/dimens.xml
index a166b098fa0b9..e8f7b353f7a9e 100644
--- a/packages/SystemUI/res-keyguard/values/dimens.xml
+++ b/packages/SystemUI/res-keyguard/values/dimens.xml
@@ -94,5 +94,6 @@
     <!-- additional offset for clock switch area items -->
     <dimen name="clock_padding_start">28dp</dimen>
     <dimen name="below_clock_padding_start">32dp</dimen>
-    <dimen name="below_clock_padding_start_extra">36dp</dimen>
+    <dimen name="below_clock_padding_end">16dp</dimen>
+    <dimen name="below_clock_padding_start_icons">28dp</dimen>
 </resources>
diff --git a/packages/SystemUI/src/com/android/keyguard/KeyguardClockSwitchController.java b/packages/SystemUI/src/com/android/keyguard/KeyguardClockSwitchController.java
index 28534d380c1f7..3e084b7d692a8 100644
--- a/packages/SystemUI/src/com/android/keyguard/KeyguardClockSwitchController.java
+++ b/packages/SystemUI/src/com/android/keyguard/KeyguardClockSwitchController.java
@@ -183,9 +183,11 @@ public class KeyguardClockSwitchController extends ViewController<KeyguardClockS
             lp.addRule(RelativeLayout.BELOW, R.id.lockscreen_clock_view);
 
             mView.addView(mSmartspaceView, ksaIndex, lp);
-            int padding = getContext().getResources()
+            int startPadding = getContext().getResources()
                     .getDimensionPixelSize(R.dimen.below_clock_padding_start);
-            mSmartspaceView.setPadding(padding, 0, padding, 0);
+            int endPadding = getContext().getResources()
+                    .getDimensionPixelSize(R.dimen.below_clock_padding_end);
+            mSmartspaceView.setPaddingRelative(startPadding, 0, endPadding, 0);
 
             // ... but above the large clock
             lp = new RelativeLayout.LayoutParams(MATCH_PARENT, WRAP_CONTENT);