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

Unified Diff: chrome/android/java/res/layout/new_tab_page_snippets_header.xml

Issue 2230473002: Refactor and extend SnippetsBridge for multi-section support (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@categoryinfo
Patch Set: Rename SnippetsObserver to ContentSuggestionsSourceObserver Created 4 years, 4 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/res/layout/new_tab_page_snippets_header.xml
diff --git a/chrome/android/java/res/layout/new_tab_page_snippets_header.xml b/chrome/android/java/res/layout/new_tab_page_snippets_header.xml
index f13a91d90ecce789b0e0726f7608551b27a9d076..82be3943619f6e13b76c6da0f7981301ffb2abdc 100644
--- a/chrome/android/java/res/layout/new_tab_page_snippets_header.xml
+++ b/chrome/android/java/res/layout/new_tab_page_snippets_header.xml
@@ -5,10 +5,9 @@
<TextView
xmlns:android="http://schemas.android.com/apk/res/android"
- android:id="@+id/snippets_list_header"
+ android:id="@+id/suggestions_section_header_text"
Michael van Ouwerkerk 2016/08/09 13:52:20 nit: omit the _text suffix, it seems redundant
Philipp Keck 2016/08/09 16:15:06 Done.
android:layout_width="match_parent"
android:layout_height="wrap_content"
- android:text="@string/snippets_header"
android:textColor="@color/snippets_list_header_text_color"
android:textSize="12sp"
android:paddingStart="@dimen/snippets_padding_and_peeking_card_height"

Powered by Google App Engine
This is Rietveld 408576698