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

Unified Diff: chrome/android/java/src/org/chromium/chrome/browser/ntp/cards/MarginResizer.java

Issue 2167973004: Revert of 📰 Adjust the card display depending on the screen width. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@ZineTabletUI
Patch Set: Created 4 years, 5 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/ntp/cards/MarginResizer.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/ntp/cards/MarginResizer.java b/chrome/android/java/src/org/chromium/chrome/browser/ntp/cards/MarginResizer.java
deleted file mode 100644
index 71591da535c085c5e3b9d40c186959fc37657c46..0000000000000000000000000000000000000000
--- a/chrome/android/java/src/org/chromium/chrome/browser/ntp/cards/MarginResizer.java
+++ /dev/null
@@ -1,41 +0,0 @@
-// Copyright 2016 The Chromium Authors. All rights reserved.
-// Use of this source code is governed by a BSD-style license that can be
-// found in the LICENSE file.
-
-package org.chromium.chrome.browser.ntp.cards;
-
-import android.view.View;
-import android.view.ViewGroup.MarginLayoutParams;
-
-import org.chromium.chrome.R;
-import org.chromium.chrome.browser.ntp.DisplayStyleObserver;
-import org.chromium.chrome.browser.ntp.UiConfig;
-
-/**
- * Adds lateral margins to the view when the display style is {@link UiConfig#DISPLAY_STYLE_WIDE}.
- */
-public class MarginResizer implements DisplayStyleObserver {
- private final int mWideMarginSizePixels;
- private final View mView;
-
- public static DisplayStyleObserverAdapter createWithViewAdapter(View view, UiConfig config) {
- return new DisplayStyleObserverAdapter(view, config, new MarginResizer(view));
- }
-
- public MarginResizer(View view) {
- mView = view;
- mWideMarginSizePixels =
- view.getResources().getDimensionPixelSize(R.dimen.ntp_wide_card_lateral_margins);
- }
-
- @Override
- public void onDisplayStyleChanged(@UiConfig.DisplayStyle int newDisplayStyle) {
- MarginLayoutParams layoutParams = (MarginLayoutParams) mView.getLayoutParams();
- if (newDisplayStyle == UiConfig.DISPLAY_STYLE_WIDE) {
- layoutParams.setMargins(mWideMarginSizePixels, layoutParams.topMargin,
- mWideMarginSizePixels, layoutParams.bottomMargin);
- } else {
- layoutParams.setMargins(0, layoutParams.topMargin, 0, layoutParams.bottomMargin);
- }
- }
-}

Powered by Google App Engine
This is Rietveld 408576698