From 2e537a87d990044f865d071b25fb6bb3e378ebde Mon Sep 17 00:00:00 2001
From: Miranda Kephart <mkephart@google.com>
Date: Tue, 15 Jun 2021 15:36:26 +0000
Subject: [PATCH] Use sp for screenshot button icon size

The icons behave more like text, and can be mixed with text, so
use sp to align the button sizes regardless of whether or not each
button has text.

Bug: 191147490
Fix: 191147490
Test: manual, visual verification (on e.g. Settings to get the
Capture More button)

Change-Id: I813fd5761dc8b3c42f150a61555514c22e33dacc
---
 packages/SystemUI/res/values/dimens.xml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/packages/SystemUI/res/values/dimens.xml b/packages/SystemUI/res/values/dimens.xml
index 0022039a78ca3..8a769c1e767e0 100644
--- a/packages/SystemUI/res/values/dimens.xml
+++ b/packages/SystemUI/res/values/dimens.xml
@@ -347,7 +347,7 @@
     <!-- Padding to make tappable chip height 48dp (18+11+11+4+4) -->
     <dimen name="screenshot_action_chip_margin_vertical">4dp</dimen>
     <dimen name="screenshot_action_chip_padding_vertical">11dp</dimen>
-    <dimen name="screenshot_action_chip_icon_size">18dp</dimen>
+    <dimen name="screenshot_action_chip_icon_size">18sp</dimen>
     <!-- Padding on each side of the icon for icon-only chips -->
     <dimen name="screenshot_action_chip_icon_only_padding_horizontal">14dp</dimen>
     <!-- Padding at the edges of the chip for icon-and-text chips -->