Skip to content

Commit 564c03d

Browse files
dsandlerAndroid (Google) Code Review
authored andcommitted
Merge "Really hide the settings button on tablets." into jb-mr1-dev
2 parents 39c6526 + 7382a02 commit 564c03d

File tree

2 files changed

+7
-3
lines changed

2 files changed

+7
-3
lines changed

packages/SystemUI/res/layout/status_bar_expanded_header.xml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -77,16 +77,16 @@
7777
<ImageView android:id="@+id/clear_all_button"
7878
android:layout_width="50dp"
7979
android:layout_height="50dp"
80-
android:layout_marginRight="12dp"
8180
android:scaleType="center"
8281
android:src="@drawable/ic_notify_clear"
8382
android:background="@drawable/ic_notify_button_bg"
8483
android:contentDescription="@string/accessibility_clear_all"
8584
/>
8685

87-
<FrameLayout
86+
<FrameLayout android:id="@+id/settings_button_holder"
8887
android:layout_width="50dp"
8988
android:layout_height="50dp"
89+
android:layout_marginLeft="12dp"
9090
>
9191
<ImageView android:id="@+id/settings_button"
9292
android:layout_width="50dp"

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

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -452,7 +452,11 @@ public boolean onTouch(View v, MotionEvent event) {
452452
mSettingsButton.setVisibility(View.VISIBLE);
453453
} else {
454454
// there is a settings panel, but it's on the other side of the (large) screen
455-
mSettingsButton.setVisibility(View.GONE);
455+
final View buttonHolder = mStatusBarWindow.findViewById(
456+
R.id.settings_button_holder);
457+
if (buttonHolder != null) {
458+
buttonHolder.setVisibility(View.GONE);
459+
}
456460
}
457461
} else {
458462
// no settings panel, go straight to settings

0 commit comments

Comments
 (0)