Skip to content

Commit 27a0ee3

Browse files
Evan LairdAndroid (Google) Code Review
authored andcommitted
Merge "DO NOT MERGE: Use status_bar_padding_top" into qt-qpr1-dev
2 parents 65964e0 + bc95ab7 commit 27a0ee3

5 files changed

Lines changed: 33 additions & 6 deletions

File tree

packages/SystemUI/res/layout/keyguard_status_bar.xml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -30,6 +30,7 @@
3030
android:id="@+id/status_icon_area"
3131
android:layout_width="wrap_content"
3232
android:layout_height="match_parent"
33+
android:paddingTop="@dimen/status_bar_padding_top"
3334
android:layout_alignParentEnd="true"
3435
android:gravity="center_vertical|end" >
3536
<FrameLayout android:id="@+id/system_icons_container"
@@ -66,6 +67,7 @@
6667
android:id="@+id/keyguard_carrier_text"
6768
android:layout_width="match_parent"
6869
android:layout_height="match_parent"
70+
android:paddingTop="@dimen/status_bar_padding_top"
6971
android:layout_marginStart="@dimen/keyguard_carrier_text_margin"
7072
android:layout_toStartOf="@id/system_icons_container"
7173
android:gravity="center_vertical"

packages/SystemUI/res/layout/quick_status_bar_header_system_icons.xml

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,8 @@
2525
android:orientation="horizontal"
2626
android:clickable="true"
2727
android:paddingStart="@dimen/status_bar_padding_start"
28-
android:paddingEnd="@dimen/status_bar_padding_end" >
28+
android:paddingEnd="@dimen/status_bar_padding_end"
29+
android:paddingTop="@dimen/status_bar_padding_top" >
2930

3031
<com.android.systemui.statusbar.policy.Clock
3132
android:id="@+id/clock"

packages/SystemUI/res/layout/system_icons.xml

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,6 @@
2626
android:layout_weight="1"
2727
android:layout_height="match_parent"
2828
android:paddingEnd="@dimen/signal_cluster_battery_padding"
29-
android:paddingTop="@dimen/status_bar_padding_top"
3029
android:gravity="center_vertical"
3130
android:orientation="horizontal"/>
3231

@@ -36,4 +35,4 @@
3635
android:clipToPadding="false"
3736
android:clipChildren="false"
3837
systemui:textAppearance="@style/TextAppearance.StatusBar.Clock" />
39-
</LinearLayout>
38+
</LinearLayout>

packages/SystemUI/src/com/android/systemui/qs/QuickStatusBarHeader.java

Lines changed: 8 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -402,10 +402,15 @@ public WindowInsets onApplyWindowInsets(WindowInsets insets) {
402402
cutout, getDisplay());
403403
if (padding == null) {
404404
mSystemIconsView.setPaddingRelative(
405-
getResources().getDimensionPixelSize(R.dimen.status_bar_padding_start), 0,
406-
getResources().getDimensionPixelSize(R.dimen.status_bar_padding_end), 0);
405+
getResources().getDimensionPixelSize(R.dimen.status_bar_padding_start),
406+
getResources().getDimensionPixelSize(R.dimen.status_bar_padding_top),
407+
getResources().getDimensionPixelSize(R.dimen.status_bar_padding_end),
408+
0);
407409
} else {
408-
mSystemIconsView.setPadding(padding.first, 0, padding.second, 0);
410+
mSystemIconsView.setPadding(
411+
padding.first,
412+
getResources().getDimensionPixelSize(R.dimen.status_bar_padding_top),
413+
padding.second, 0);
409414

410415
}
411416
return super.onApplyWindowInsets(insets);

packages/SystemUI/src/com/android/systemui/statusbar/phone/PhoneStatusBarView.java

Lines changed: 20 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -131,6 +131,7 @@ protected void onDetachedFromWindow() {
131131
@Override
132132
protected void onConfigurationChanged(Configuration newConfig) {
133133
super.onConfigurationChanged(newConfig);
134+
updateResources();
134135

135136
// May trigger cutout space layout-ing
136137
if (updateOrientationAndCutout(newConfig.orientation)) {
@@ -297,9 +298,28 @@ public void updateResources() {
297298
mCutoutSideNudge = getResources().getDimensionPixelSize(
298299
R.dimen.display_cutout_margin_consumption);
299300

301+
boolean isRtl = getLayoutDirection() == View.LAYOUT_DIRECTION_RTL;
302+
303+
int statusBarPaddingTop = getResources().getDimensionPixelSize(
304+
R.dimen.status_bar_padding_top);
305+
int statusBarPaddingStart = getResources().getDimensionPixelSize(
306+
R.dimen.status_bar_padding_start);
307+
int statusBarPaddingEnd = getResources().getDimensionPixelSize(
308+
R.dimen.status_bar_padding_end);
309+
300310
ViewGroup.LayoutParams layoutParams = getLayoutParams();
301311
mStatusBarHeight = getResources().getDimensionPixelSize(R.dimen.status_bar_height);
302312
layoutParams.height = mStatusBarHeight;
313+
314+
View sbContents = findViewById(R.id.status_bar_contents);
315+
sbContents.setPadding(
316+
isRtl ? statusBarPaddingEnd : statusBarPaddingStart,
317+
statusBarPaddingTop,
318+
isRtl ? statusBarPaddingStart : statusBarPaddingEnd,
319+
0);
320+
321+
findViewById(R.id.notification_lights_out).setPadding(0, statusBarPaddingStart, 0, 0);
322+
303323
setLayoutParams(layoutParams);
304324
}
305325

0 commit comments

Comments
 (0)