Android/frameworks_base
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge "Correct RTL support of Smart Actions" into sc-qpr1-dev am: a69d3d442c

Browse Source 
Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/base/+/15661106

Change-Id: Ib67a33aaf5d0c42ce9b7173bd9b4290ea4a51471
This commit is contained in:
Jeff DeCew
2021-09-07 17:53:55 +00:00
committed by Automerger Merge Worker
parent 29745061fa a69d3d442c
commit 55b7e82660
5 changed files with 18 additions and 28 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
packages/SystemUI/res/layout/smart_action_button.xml
View File

@@ -29,8 +29,8 @@
android:textSize="@dimen/smart_reply_button_font_size" android:textSize="@dimen/smart_reply_button_font_size"
android:lineSpacingExtra="@dimen/smart_reply_button_line_spacing_extra" android:lineSpacingExtra="@dimen/smart_reply_button_line_spacing_extra"
android:textColor="@color/smart_reply_button_text" android:textColor="@color/smart_reply_button_text"
android:paddingLeft="@dimen/smart_reply_button_action_padding_left" android:paddingStart="@dimen/smart_reply_button_action_padding_left"
android:paddingRight="@dimen/smart_reply_button_padding_horizontal" android:paddingEnd="@dimen/smart_reply_button_padding_horizontal"
android:drawablePadding="@dimen/smart_action_button_icon_padding" android:drawablePadding="@dimen/smart_action_button_icon_padding"
android:textStyle="normal" android:textStyle="normal"
android:ellipsize="none"/> android:ellipsize="none"/>

4
packages/SystemUI/res/layout/smart_reply_button.xml
View File

@@ -31,7 +31,7 @@
android:textSize="@dimen/smart_reply_button_font_size" android:textSize="@dimen/smart_reply_button_font_size"
android:lineSpacingExtra="@dimen/smart_reply_button_line_spacing_extra" android:lineSpacingExtra="@dimen/smart_reply_button_line_spacing_extra"
android:textColor="@color/smart_reply_button_text" android:textColor="@color/smart_reply_button_text"
android:paddingLeft="@dimen/smart_reply_button_padding_horizontal" android:paddingStart="@dimen/smart_reply_button_padding_horizontal"
android:paddingRight="@dimen/smart_reply_button_padding_horizontal" android:paddingEnd="@dimen/smart_reply_button_padding_horizontal"
android:textStyle="normal" android:textStyle="normal"
android:ellipsize="none"/> android:ellipsize="none"/>

2
packages/SystemUI/src/com/android/systemui/statusbar/policy/SmartReplyStateInflater.kt
View File

@@ -311,7 +311,7 @@ interface SmartActionInflater {
setBounds(0, 0, newIconSize, newIconSize) setBounds(0, 0, newIconSize, newIconSize)
} }
// Add the action icon to the Smart Action button. // Add the action icon to the Smart Action button.
setCompoundDrawables(iconDrawable, null, null, null) setCompoundDrawablesRelative(iconDrawable, null, null, null)
val onClickListener = View.OnClickListener { val onClickListener = View.OnClickListener {
onSmartActionClick(entry, smartActions, actionIndex, action) onSmartActionClick(entry, smartActions, actionIndex, action)

18
packages/SystemUI/src/com/android/systemui/statusbar/policy/SmartReplyView.java
View File

@@ -503,15 +503,15 @@ public class SmartReplyView extends ViewGroup {
} }
/** /**
* Returns the combined width of the left drawable (the action icon) and the padding between the * Returns the combined width of the start drawable (the action icon) and the padding between
* drawable and the button text. * the drawable and the button text.
*/ */
private int getLeftCompoundDrawableWidthWithPadding(Button button) { private int getStartCompoundDrawableWidthWithPadding(Button button) {
Drawable[] drawables = button.getCompoundDrawables(); Drawable[] drawables = button.getCompoundDrawablesRelative();
Drawable leftDrawable = drawables[0]; Drawable startDrawable = drawables[0];
if (leftDrawable == null) return 0; if (startDrawable == null) return 0;
return leftDrawable.getBounds().width() + button.getCompoundDrawablePadding(); return startDrawable.getBounds().width() + button.getCompoundDrawablePadding();
} }
private int squeezeButtonToTextWidth(Button button, int heightMeasureSpec, int textWidth) { private int squeezeButtonToTextWidth(Button button, int heightMeasureSpec, int textWidth) {
@@ -520,8 +520,8 @@ public class SmartReplyView extends ViewGroup {
// Re-measure the squeezed smart reply button. // Re-measure the squeezed smart reply button.
clearLayoutLineCount(button); clearLayoutLineCount(button);
final int widthMeasureSpec = MeasureSpec.makeMeasureSpec( final int widthMeasureSpec = MeasureSpec.makeMeasureSpec(
button.getPaddingLeft() + button.getPaddingRight() + textWidth button.getPaddingStart() + button.getPaddingEnd() + textWidth
+ getLeftCompoundDrawableWidthWithPadding(button), MeasureSpec.AT_MOST); + getStartCompoundDrawableWidthWithPadding(button), MeasureSpec.AT_MOST);
button.measure(widthMeasureSpec, heightMeasureSpec); button.measure(widthMeasureSpec, heightMeasureSpec);
final int newWidth = button.getMeasuredWidth(); final int newWidth = button.getMeasuredWidth();

18
packages/SystemUI/tests/src/com/android/systemui/statusbar/policy/SmartReplyViewTest.java
View File

@@ -581,8 +581,6 @@ public class SmartReplyViewTest extends SysuiTestCase {
// devices. // devices.
layout.setBaselineAligned(false); layout.setBaselineAligned(false);
final boolean isRtl = mView.getLayoutDirection() == View.LAYOUT_DIRECTION_RTL;
// Add smart replies // Add smart replies
Button previous = null; Button previous = null;
SmartReplyView.SmartReplies smartReplies = SmartReplyView.SmartReplies smartReplies =
@@ -602,11 +600,7 @@ public class SmartReplyViewTest extends SysuiTestCase {
if (previous != null) { if (previous != null) {
ViewGroup.MarginLayoutParams lp = ViewGroup.MarginLayoutParams lp =
(ViewGroup.MarginLayoutParams) previous.getLayoutParams(); (ViewGroup.MarginLayoutParams) previous.getLayoutParams();
if (isRtl) { lp.setMarginEnd(mSpacing);
lp.leftMargin = mSpacing;
} else {
lp.rightMargin = mSpacing;
}
} }
layout.addView(current); layout.addView(current);
previous = current; previous = current;
@@ -630,11 +624,7 @@ public class SmartReplyViewTest extends SysuiTestCase {
if (previous != null) { if (previous != null) {
ViewGroup.MarginLayoutParams lp = ViewGroup.MarginLayoutParams lp =
(ViewGroup.MarginLayoutParams) previous.getLayoutParams(); (ViewGroup.MarginLayoutParams) previous.getLayoutParams();
if (isRtl) { lp.setMarginEnd(mSpacing);
lp.leftMargin = mSpacing;
} else {
lp.rightMargin = mSpacing;
}
} }
layout.addView(current); layout.addView(current);
previous = current; previous = current;
@@ -933,8 +923,8 @@ public class SmartReplyViewTest extends SysuiTestCase {
.collect(Collectors.toList()); .collect(Collectors.toList());
Button singleLineButton = buttons.get(0); Button singleLineButton = buttons.get(0);
Button doubleLineButton = buttons.get(1); Button doubleLineButton = buttons.get(1);
Drawable singleLineDrawable = singleLineButton.getCompoundDrawables()[0]; // left drawable Drawable singleLineDrawable = singleLineButton.getCompoundDrawablesRelative()[0]; // start
Drawable doubleLineDrawable = doubleLineButton.getCompoundDrawables()[0]; // left drawable Drawable doubleLineDrawable = doubleLineButton.getCompoundDrawablesRelative()[0]; // start
assertEquals(singleLineDrawable.getBounds().width(), assertEquals(singleLineDrawable.getBounds().width(),
doubleLineDrawable.getBounds().width()); doubleLineDrawable.getBounds().width());
assertEquals(singleLineDrawable.getBounds().height(), assertEquals(singleLineDrawable.getBounds().height(),