1
0
Fork 0

No issue: Move dp's to dimens.xml

master
Hakkı Kaan Çalışkan 2020-07-15 20:05:43 +03:00 committed by Emily Kager
parent b867e0de45
commit b997fba953
7 changed files with 28 additions and 21 deletions

View File

@ -17,8 +17,8 @@
android:fontFamily="@font/metropolis_bold" android:fontFamily="@font/metropolis_bold"
android:layout_width="wrap_content" android:layout_width="wrap_content"
android:layout_height="@dimen/tracking_protection_item_height" android:layout_height="@dimen/tracking_protection_item_height"
android:layout_marginStart="72dp" android:layout_marginStart="@dimen/top_bar_alignment_margin_start"
android:layout_marginEnd="16dp" android:layout_marginEnd="@dimen/tracking_protection_item_margin_end"
android:paddingStart="0dp" android:paddingStart="0dp"
android:paddingEnd="0dp" android:paddingEnd="0dp"
android:text="@string/enhanced_tracking_protection_blocked" /> android:text="@string/enhanced_tracking_protection_blocked" />

View File

@ -6,7 +6,7 @@
xmlns:app="http://schemas.android.com/apk/res-auto" xmlns:app="http://schemas.android.com/apk/res-auto"
xmlns:tools="http://schemas.android.com/tools" xmlns:tools="http://schemas.android.com/tools"
android:layout_width="match_parent" android:layout_width="match_parent"
android:layout_height="48dp"> android:layout_height="@dimen/locale_item_height">
<ImageView <ImageView
android:id="@+id/locale_selected_icon" android:id="@+id/locale_selected_icon"

View File

@ -14,10 +14,10 @@
<FrameLayout <FrameLayout
android:id="@+id/favicon_wrapper" android:id="@+id/favicon_wrapper"
android:layout_width="40dp" android:layout_width="@dimen/history_favicon_width_height"
android:layout_height="40dp" android:layout_height="@dimen/history_favicon_width_height"
android:padding="4dp" android:padding="@dimen/saved_logins_item_padding"
android:layout_marginStart="16dp" android:layout_marginStart="@dimen/saved_logins_item_margin_start"
android:background="@drawable/top_sites_background" android:background="@drawable/top_sites_background"
android:layout_gravity="center" android:layout_gravity="center"
android:importantForAccessibility="noHideDescendants" android:importantForAccessibility="noHideDescendants"
@ -26,8 +26,8 @@
app:layout_constraintBottom_toBottomOf="parent"> app:layout_constraintBottom_toBottomOf="parent">
<ImageView <ImageView
android:id="@+id/favicon_image" android:id="@+id/favicon_image"
android:layout_width="24dp" android:layout_width="@dimen/preference_icon_drawable_size"
android:layout_height="24dp" android:layout_height="@dimen/preference_icon_drawable_size"
android:layout_gravity="center" android:layout_gravity="center"
android:adjustViewBounds="true" android:adjustViewBounds="true"
android:importantForAccessibility="no" android:importantForAccessibility="no"
@ -38,8 +38,8 @@
android:id="@+id/webAddressView" android:id="@+id/webAddressView"
android:layout_width="0dp" android:layout_width="0dp"
android:layout_height="wrap_content" android:layout_height="wrap_content"
android:layout_marginStart="16dp" android:layout_marginStart="@dimen/saved_logins_item_margin_start"
android:layout_marginEnd="48dp" android:layout_marginEnd="@dimen/saved_logins_item_margin_end"
android:ellipsize="middle" android:ellipsize="middle"
android:singleLine="true" android:singleLine="true"
android:textAppearance="?android:attr/textAppearanceListItem" android:textAppearance="?android:attr/textAppearanceListItem"
@ -55,8 +55,8 @@
android:id="@+id/usernameView" android:id="@+id/usernameView"
android:layout_width="0dp" android:layout_width="0dp"
android:layout_height="wrap_content" android:layout_height="wrap_content"
android:layout_marginStart="16dp" android:layout_marginStart="@dimen/saved_logins_item_margin_start"
android:layout_marginEnd="48dp" android:layout_marginEnd="@dimen/saved_logins_item_margin_end"
android:ellipsize="middle" android:ellipsize="middle"
android:singleLine="true" android:singleLine="true"
android:textColor="?secondaryText" android:textColor="?secondaryText"

View File

@ -8,7 +8,7 @@
style="?android:attr/listViewStyle" style="?android:attr/listViewStyle"
android:layout_width="match_parent" android:layout_width="match_parent"
android:layout_height="wrap_content" android:layout_height="wrap_content"
android:layout_marginStart="72dp" android:layout_marginStart="@dimen/top_bar_alignment_margin_start"
android:layout_marginTop="16dp" android:layout_marginTop="16dp"
android:gravity="start|center_vertical" android:gravity="start|center_vertical"
android:textAlignment="viewStart" android:textAlignment="viewStart"

View File

@ -29,7 +29,7 @@
<TextView <TextView
android:id="@+id/title" android:id="@+id/title"
android:layout_width="0dp" android:layout_width="0dp"
android:layout_marginStart="8dp" android:layout_marginStart="@dimen/radiobutton_preference_margin_start"
android:layout_height="wrap_content" android:layout_height="wrap_content"
android:gravity="center|start" android:gravity="center|start"
android:layout_marginEnd="@dimen/radio_button_preference_horizontal" android:layout_marginEnd="@dimen/radio_button_preference_horizontal"

View File

@ -11,8 +11,8 @@
<ImageView <ImageView
android:id="@+id/trackingProtectionCategoryIcon" android:id="@+id/trackingProtectionCategoryIcon"
android:layout_width="24dp" android:layout_width="@dimen/preference_icon_drawable_size"
android:layout_height="24dp" android:layout_height="@dimen/preference_icon_drawable_size"
android:layout_marginStart="@dimen/library_item_icon_margin_horizontal" android:layout_marginStart="@dimen/library_item_icon_margin_horizontal"
android:layout_marginTop="@dimen/library_item_icon_margin_vertical" android:layout_marginTop="@dimen/library_item_icon_margin_vertical"
android:layout_marginEnd="@dimen/library_item_icon_margin_horizontal" android:layout_marginEnd="@dimen/library_item_icon_margin_horizontal"
@ -27,8 +27,8 @@
android:id="@+id/trackingProtectionCategoryTitle" android:id="@+id/trackingProtectionCategoryTitle"
android:layout_width="0dp" android:layout_width="0dp"
android:layout_height="wrap_content" android:layout_height="wrap_content"
android:layout_marginStart="32dp" android:layout_marginStart="@dimen/radio_button_drawable_padding"
android:layout_marginTop="12dp" android:layout_marginTop="@dimen/about_list_margin_top"
android:layout_marginEnd="@dimen/library_item_icon_margin_horizontal" android:layout_marginEnd="@dimen/library_item_icon_margin_horizontal"
android:clickable="false" android:clickable="false"
android:textAppearance="@style/ListItemTextStyle" android:textAppearance="@style/ListItemTextStyle"
@ -44,9 +44,9 @@
android:id="@+id/trackingProtectionCategoryItemDescription" android:id="@+id/trackingProtectionCategoryItemDescription"
android:layout_width="0dp" android:layout_width="0dp"
android:layout_height="wrap_content" android:layout_height="wrap_content"
android:layout_marginStart="32dp" android:layout_marginStart="@dimen/radio_button_drawable_padding"
android:layout_marginEnd="@dimen/library_item_icon_margin_horizontal" android:layout_marginEnd="@dimen/library_item_icon_margin_horizontal"
android:layout_marginBottom="18dp" android:layout_marginBottom="@dimen/tracking_protection_item_margin_bottom"
android:clickable="false" android:clickable="false"
android:textColor="?attr/secondaryText" android:textColor="?attr/secondaryText"
app:layout_constraintBottom_toBottomOf="parent" app:layout_constraintBottom_toBottomOf="parent"

View File

@ -50,11 +50,14 @@
<!--Preferences--> <!--Preferences-->
<dimen name="checkbox_preference_padding_vertical">12dp</dimen> <dimen name="checkbox_preference_padding_vertical">12dp</dimen>
<dimen name="radiobutton_preference_margin_start">8dp</dimen>
<dimen name="top_bar_alignment_margin_start">72dp</dimen> <dimen name="top_bar_alignment_margin_start">72dp</dimen>
<!--Quick Settings--> <!--Quick Settings-->
<dimen name="quicksettings_item_height">28dp</dimen> <dimen name="quicksettings_item_height">28dp</dimen>
<dimen name="tracking_protection_item_height">48dp</dimen> <dimen name="tracking_protection_item_height">48dp</dimen>
<dimen name="tracking_protection_item_margin_end">16dp</dimen>
<dimen name="tracking_protection_item_margin_bottom">18dp</dimen>
<dimen name="design_quick_action_sheet_peek_height_min">64dp</dimen> <dimen name="design_quick_action_sheet_peek_height_min">64dp</dimen>
@ -116,6 +119,7 @@
<dimen name="locale_item_text_margin_gone_start">72dp</dimen> <dimen name="locale_item_text_margin_gone_start">72dp</dimen>
<dimen name="locale_item_title_size">16sp</dimen> <dimen name="locale_item_title_size">16sp</dimen>
<dimen name="locale_item_subtitle_size">12sp</dimen> <dimen name="locale_item_subtitle_size">12sp</dimen>
<dimen name="locale_item_height">48dp</dimen>
<!--Migration Activity--> <!--Migration Activity-->
<dimen name="migration_margin_horizontal_large">20dp</dimen> <dimen name="migration_margin_horizontal_large">20dp</dimen>
@ -157,6 +161,9 @@
<dimen name="saved_logins_sort_menu_dropdown_chevron_icon_size">12dp</dimen> <dimen name="saved_logins_sort_menu_dropdown_chevron_icon_size">12dp</dimen>
<dimen name="saved_logins_detail_menu_vertical_padding">5dp</dimen> <dimen name="saved_logins_detail_menu_vertical_padding">5dp</dimen>
<dimen name="saved_logins_end_icon_drawable_padding">16dp</dimen> <dimen name="saved_logins_end_icon_drawable_padding">16dp</dimen>
<dimen name="saved_logins_item_padding">4dp</dimen>
<dimen name="saved_logins_item_margin_start">16dp</dimen>
<dimen name="saved_logins_item_margin_end">48dp</dimen>
<!-- a11y --> <!-- a11y -->
<dimen name="accessibility_min_height">48dp</dimen> <dimen name="accessibility_min_height">48dp</dimen>