Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(3220)

Unified Diff: chrome/android/java/src/org/chromium/chrome/browser/widget/selection/SelectableListToolbar.java

Issue 2854643002: [Android] Width constrain bookmarks and downloads UIs (Closed)
Patch Set: Changes from dfalcantara@ review Created 3 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/android/java/src/org/chromium/chrome/browser/widget/selection/SelectableListToolbar.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/widget/selection/SelectableListToolbar.java b/chrome/android/java/src/org/chromium/chrome/browser/widget/selection/SelectableListToolbar.java
index add383a00e074c0475685968f909d117e7715398..0ef88875a5e8dd210878417c92c2f4e25cc3af96 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/widget/selection/SelectableListToolbar.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/widget/selection/SelectableListToolbar.java
@@ -363,6 +363,8 @@ public class SelectableListToolbar<E> extends Toolbar implements SelectionObserv
setNavigationIcon(iconResId);
}
setNavigationContentDescription(contentDescriptionId);
+
+ updateDisplayStyleIfNecessary();
}
/**
@@ -387,8 +389,9 @@ public class SelectableListToolbar<E> extends Toolbar implements SelectionObserv
public void hideSearchView() {
assert mHasSearchView;
- mIsSearching = false;
+ if (!mIsSearching) return;
+ mIsSearching = false;
mSearchEditText.setText("");
UiUtils.hideKeyboard(mSearchEditText);
showNormalView();
@@ -434,8 +437,9 @@ public class SelectableListToolbar<E> extends Toolbar implements SelectionObserv
*
* @param wideDisplayLateralOffsetPx The offset to use for the lateral padding when in
* {@link HorizontalDisplayStyle#WIDE}.
+ * @param uiConfig The UiConfig used to observe display style changes.
*/
- public void setHasWideDisplayStyle(int wideDisplayLateralOffsetPx, UiConfig uiConfig) {
+ public void configureWideDisplayStyle(int wideDisplayLateralOffsetPx, UiConfig uiConfig) {
mWideDisplayLateralOffsetPx = wideDisplayLateralOffsetPx;
mDefaultTitleMarginStartPx = getTitleMarginStart();
mWideDisplayNavButtonOffsetPx =
@@ -464,7 +468,8 @@ public class SelectableListToolbar<E> extends Toolbar implements SelectionObserv
// The title and nav buttons are inset in the normal display style. In the wide display
// style they should be aligned with the starting edge of the list elements.
- if (mIsSearching || mIsSelectionEnabled) {
+ if (mIsSearching || mIsSelectionEnabled
+ || mNavigationButton != NAVIGATION_BUTTON_NONE) {
paddingStartOffset += mWideDisplayNavButtonOffsetPx;
} else {
paddingStartOffset -= mDefaultTitleMarginStartPx;

Powered by Google App Engine
This is Rietveld 408576698