diff options
author | Steve Kondik <shade@chemlab.org> | 2012-08-10 06:04:48 +0400 |
---|---|---|
committer | Gerrit Code Review <gerrit@review.cyanogenmod.com> | 2012-08-10 06:04:48 +0400 |
commit | 384ecb88d23f088e40407de31de0a91b9cc105ec (patch) | |
tree | 31a14e8062fb8a80cf029ebb97f771b5d60c7316 | |
parent | fd5bbdab3224478f3e67597d6d65afb3bae3f4ae (diff) | |
parent | c02ae2457a11e4631f4593480c4905424d024018 (diff) | |
download | frameworks_base-384ecb88d23f088e40407de31de0a91b9cc105ec.zip frameworks_base-384ecb88d23f088e40407de31de0a91b9cc105ec.tar.gz frameworks_base-384ecb88d23f088e40407de31de0a91b9cc105ec.tar.bz2 |
Merge "Fix 5761: Power widget bar widens if >6 toggle buttons enabled" into ics
-rw-r--r-- | packages/SystemUI/src/com/android/systemui/statusbar/powerwidget/PowerWidget.java | 17 |
1 files changed, 11 insertions, 6 deletions
diff --git a/packages/SystemUI/src/com/android/systemui/statusbar/powerwidget/PowerWidget.java b/packages/SystemUI/src/com/android/systemui/statusbar/powerwidget/PowerWidget.java index 5f8f91f..1fc48d9 100644 --- a/packages/SystemUI/src/com/android/systemui/statusbar/powerwidget/PowerWidget.java +++ b/packages/SystemUI/src/com/android/systemui/statusbar/powerwidget/PowerWidget.java @@ -272,11 +272,6 @@ public class PowerWidget extends FrameLayout { mScrollView.setFadingEdgeLength(mContext.getResources().getDisplayMetrics().widthPixels / LAYOUT_SCROLL_BUTTON_THRESHOLD); mScrollView.setScrollBarStyle(View.SCROLLBARS_INSIDE_INSET); mScrollView.setOverScrollMode(View.OVER_SCROLL_NEVER); - // set the padding on the linear layout to the size of our scrollbar, so we don't have them overlap - mButtonLayout.setPadding(mButtonLayout.getPaddingLeft(), - mButtonLayout.getPaddingTop(), - mButtonLayout.getPaddingRight(), - mScrollView.getVerticalScrollbarWidth()); mScrollView.addView(mButtonLayout, WIDGET_LAYOUT_PARAMS); updateScrollbar(); addView(mScrollView, WIDGET_LAYOUT_PARAMS); @@ -370,6 +365,15 @@ public class PowerWidget extends FrameLayout { boolean hideScrollBar = Settings.System.getInt(mContext.getContentResolver(), Settings.System.EXPANDED_HIDE_SCROLLBAR, 0) == 1; mScrollView.setHorizontalScrollBarEnabled(!hideScrollBar); + + // set the padding on the linear layout to the size of our scrollbar, + // so we don't have them overlap + // need to be here for make use of EXPANDED_HIDE_SCROLLBAR, expanding or collapsing + // the space used by the scrollbar + if (mButtonLayout != null) { + mButtonLayout.setPadding(0, 0, 0, + !hideScrollBar ? mScrollView.getVerticalScrollbarWidth() : 0); + } } private void updateHapticFeedbackSetting() { @@ -486,7 +490,8 @@ public class PowerWidget extends FrameLayout { updateVisibility(); // now check for scrollbar hiding } else if(uri.equals(Settings.System.getUriFor(Settings.System.EXPANDED_HIDE_SCROLLBAR))) { - updateScrollbar(); + // Needed to remove scrollview to gain the space of the scrollable area + recreateButtonLayout(); } if (uri.equals(Settings.System.getUriFor(Settings.System.HAPTIC_FEEDBACK_ENABLED)) |