Chromium Code Reviews| OLD | NEW |
|---|---|
| 1 <?xml version="1.0" encoding="utf-8"?> | 1 <?xml version="1.0" encoding="utf-8"?> |
| 2 <!-- | 2 <!-- |
| 3 Copyright 2013 The Chromium Authors. All rights reserved. | 3 Copyright 2013 The Chromium Authors. All rights reserved. |
| 4 | 4 |
| 5 Use of this source code is governed by a BSD-style license that can be | 5 Use of this source code is governed by a BSD-style license that can be |
| 6 found in the LICENSE file. | 6 found in the LICENSE file. |
| 7 --> | 7 --> |
| 8 <LinearLayout | 8 <LinearLayout |
| 9 xmlns:android="http://schemas.android.com/apk/res/android" | 9 xmlns:android="http://schemas.android.com/apk/res/android" |
| 10 xmlns:chrome="http://schemas.android.com/apk/res-auto" | 10 xmlns:chrome="http://schemas.android.com/apk/res-auto" |
| (...skipping 11 matching lines...) Expand all Loading... | |
| 22 android:paddingEnd="@dimen/website_settings_popup_padding_sides" | 22 android:paddingEnd="@dimen/website_settings_popup_padding_sides" |
| 23 android:paddingStart="@dimen/website_settings_popup_padding_sides" > | 23 android:paddingStart="@dimen/website_settings_popup_padding_sides" > |
| 24 | 24 |
| 25 <view class="org.chromium.chrome.browser.pageinfo.WebsiteSettingsPopup$E lidedUrlTextView" | 25 <view class="org.chromium.chrome.browser.pageinfo.WebsiteSettingsPopup$E lidedUrlTextView" |
| 26 android:id="@+id/website_settings_url" | 26 android:id="@+id/website_settings_url" |
| 27 android:layout_width="match_parent" | 27 android:layout_width="match_parent" |
| 28 android:layout_height="wrap_content" | 28 android:layout_height="wrap_content" |
| 29 android:ellipsize="end" | 29 android:ellipsize="end" |
| 30 android:lineSpacingExtra="6dp" | 30 android:lineSpacingExtra="6dp" |
| 31 android:paddingTop="16dp" | 31 android:paddingTop="16dp" |
| 32 android:paddingBottom="16dp" | |
| 32 android:textAlignment="viewStart" | 33 android:textAlignment="viewStart" |
| 33 android:textColor="@color/url_emphasis_default_text" | 34 android:textColor="@color/url_emphasis_default_text" |
| 34 android:textSize="16sp"/> | 35 android:textSize="16sp"/> |
| 35 | 36 |
| 36 <TextView | 37 <TextView |
| 38 android:id="@+id/website_settings_connection_summary" | |
| 39 android:layout_width="match_parent" | |
| 40 android:layout_height="wrap_content" | |
| 41 android:lineSpacingExtra="3dp" | |
|
Ted C
2016/11/03 20:49:48
should this have paddingTop="16dp" here? and no pa
tsergeant
2016/11/03 23:01:04
I'm not sure whether this should have the extra pa
| |
| 42 android:textColor="@color/website_settings_popup_text" | |
| 43 android:textSize="16sp" | |
| 44 android:visibility="gone" /> | |
| 45 | |
| 46 <TextView | |
| 37 android:id="@+id/website_settings_connection_message" | 47 android:id="@+id/website_settings_connection_message" |
| 38 android:layout_width="match_parent" | 48 android:layout_width="match_parent" |
| 39 android:layout_height="wrap_content" | 49 android:layout_height="wrap_content" |
| 40 android:lineSpacingExtra="3dp" | 50 android:lineSpacingExtra="3dp" |
| 41 android:paddingTop="8dp" | 51 android:paddingTop="8dp" |
| 42 android:textColor="@color/website_settings_popup_text" | 52 android:textColor="@color/website_settings_popup_text" |
| 43 android:textSize="14sp" /> | 53 android:textSize="14sp" /> |
| 44 </LinearLayout> | 54 </LinearLayout> |
| 45 | 55 |
| 46 <org.chromium.ui.widget.ButtonCompat | 56 <org.chromium.ui.widget.ButtonCompat |
| (...skipping 43 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... | |
| 90 android:layout_gravity="end" | 100 android:layout_gravity="end" |
| 91 android:layout_marginEnd="@dimen/website_settings_popup_padding_sides" | 101 android:layout_marginEnd="@dimen/website_settings_popup_padding_sides" |
| 92 android:layout_marginStart="@dimen/website_settings_popup_padding_sides" | 102 android:layout_marginStart="@dimen/website_settings_popup_padding_sides" |
| 93 android:layout_marginTop="8dp" | 103 android:layout_marginTop="8dp" |
| 94 android:paddingEnd="@dimen/website_settings_popup_button_padding_sides" | 104 android:paddingEnd="@dimen/website_settings_popup_button_padding_sides" |
| 95 android:paddingStart="@dimen/website_settings_popup_button_padding_sides " | 105 android:paddingStart="@dimen/website_settings_popup_button_padding_sides " |
| 96 android:text="@string/page_info_open_online_button" | 106 android:text="@string/page_info_open_online_button" |
| 97 android:textColor="@color/light_active_color" | 107 android:textColor="@color/light_active_color" |
| 98 style="@style/ButtonCompatBorderless" /> | 108 style="@style/ButtonCompatBorderless" /> |
| 99 </LinearLayout> | 109 </LinearLayout> |
| OLD | NEW |