Chromium Code Reviews| Index: chrome/android/java/res/layout/default_search_engine_first_run_fragment.xml |
| diff --git a/chrome/android/java/res/layout/default_search_engine_first_run_fragment.xml b/chrome/android/java/res/layout/default_search_engine_first_run_fragment.xml |
| new file mode 100644 |
| index 0000000000000000000000000000000000000000..e2df7e5ab6414b522e92364316a1d4ef531dee58 |
| --- /dev/null |
| +++ b/chrome/android/java/res/layout/default_search_engine_first_run_fragment.xml |
| @@ -0,0 +1,94 @@ |
| +<?xml version="1.0" encoding="utf-8"?> |
| +<!-- Copyright 2017 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. |
| +--> |
| + |
| +<!-- Layout used to present a set of default search engines to the user. --> |
| +<LinearLayout |
| + xmlns:android="http://schemas.android.com/apk/res/android" |
| + xmlns:chrome="http://schemas.android.com/apk/res-auto" |
| + android:layout_width="match_parent" |
| + android:layout_height="match_parent" |
| + android:background="@color/signin_body_background" |
| + android:orientation="vertical" > |
| + |
| + <FrameLayout |
|
Ted C
2017/05/04 00:16:56
Why do we need this? Any reason FirstRunChooserVi
gone
2017/05/04 00:28:11
FirstRunChooserView asserts its layout parameters
|
| + android:layout_width="match_parent" |
| + android:layout_height="0dp" |
| + android:layout_weight="1" > |
| + |
| + <org.chromium.chrome.browser.firstrun.FirstRunChooserView |
| + android:layout_width="match_parent" |
| + android:layout_height="match_parent" |
| + android:fadeScrollbars="false" |
| + android:requiresFadingEdge="vertical" |
| + android:fadingEdgeLength="48dp" > |
| + |
| + <LinearLayout |
| + android:id="@+id/scrolling_content" |
| + android:layout_width="match_parent" |
| + android:layout_height="wrap_content" |
| + android:orientation="vertical"> |
| + |
| + <!-- The layout_width/layout_height is set to 16/9 dynamically in Java --> |
| + <TextView |
| + android:id="@+id/chooser_title" |
| + android:layout_width="match_parent" |
| + android:layout_height="wrap_content" |
| + android:gravity="bottom" |
| + android:paddingStart="@dimen/signin_chooser_padding" |
| + android:paddingEnd="@dimen/signin_chooser_padding" |
| + android:paddingBottom="@dimen/signin_chooser_padding" |
| + android:background="@color/signin_head_background" |
| + android:textColor="@color/default_text_color" |
| + android:textSize="@dimen/fre_title_text_size" |
| + android:text="@string/search_engine_dialog_title" /> |
| + |
| + <View |
|
Ted C
2017/05/04 00:16:56
I assume this can't use the Divider style? https:
gone
2017/05/04 00:28:11
Was intentionally not breaking from the sign in pa
|
| + android:layout_width="match_parent" |
| + android:layout_height="1dp" |
| + android:layout_marginBottom="@dimen/signin_chooser_padding" |
| + android:background="@color/signin_border_line_color" |
| + android:alpha="0.08"/> |
| + |
| + <org.chromium.chrome.browser.widget.RadioButtonLayout |
| + android:id="@+id/engine_controls" |
| + android:layout_width="match_parent" |
| + android:layout_height="wrap_content" |
| + android:layout_marginStart="12dp" |
| + android:layout_marginEnd="@dimen/signin_chooser_padding" /> |
| + </LinearLayout> |
| + </org.chromium.chrome.browser.firstrun.FirstRunChooserView> |
| + </FrameLayout> |
| + |
| + <TextView |
| + android:layout_width="match_parent" |
| + android:layout_height="wrap_content" |
| + android:text="@string/search_engine_dialog_footer" |
| + android:textColor="@color/descriptive_text_color" |
| + android:textSize="@dimen/fre_normal_text_size" |
| + android:paddingStart="@dimen/signin_chooser_padding" |
|
Ted C
2017/05/04 00:16:56
use android:padding= to replace these four?
gone
2017/05/04 00:28:11
Done.
|
| + android:paddingEnd="@dimen/signin_chooser_padding" |
| + android:paddingTop="@dimen/signin_chooser_padding" |
| + android:paddingBottom="@dimen/signin_chooser_padding" /> |
| + |
| + <!--suppress ButtonStyle --> |
| + <org.chromium.ui.widget.ButtonCompat |
| + android:id="@+id/button_primary" |
| + android:layout_width="wrap_content" |
| + android:layout_height="36dp" |
| + android:layout_gravity="end" |
| + android:layout_marginBottom="16dp" |
| + android:layout_marginStart="16dp" |
| + android:layout_marginEnd="22dp" |
| + android:paddingStart="@dimen/fre_button_padding" |
| + android:paddingEnd="@dimen/fre_button_padding" |
| + android:text="@string/ok" |
| + android:textAllCaps="true" |
| + android:textColor="@android:color/white" |
| + android:textSize="14sp" |
| + chrome:buttonColor="@color/light_active_color" |
| + chrome:buttonRaised="false" /> |
| + |
| +</LinearLayout> |