Răsfoiți Sursa

Remove material-design-dimens (#5133)

Ivan Iskandar 3 ani în urmă
părinte
comite
afd59eabbb

+ 0 - 1
app/build.gradle.kts

@@ -214,7 +214,6 @@ dependencies {
     implementation("com.github.gpanther:java-nat-sort:natural-comparator-1.1")
 
     // UI
-    implementation("com.dmitrymalkovich.android:material-design-dimens:1.4")
     implementation("com.github.dmytrodanylyk.android-process-button:library:1.0.4")
     implementation("eu.davidea:flexible-adapter:5.1.0")
     implementation("eu.davidea:flexible-adapter-ui:1.0.0")

+ 4 - 3
app/src/main/res/layout/categories_item.xml

@@ -3,13 +3,14 @@
     xmlns:app="http://schemas.android.com/apk/res-auto"
     xmlns:tools="http://schemas.android.com/tools"
     android:layout_width="match_parent"
-    android:layout_height="@dimen/material_component_lists_single_line_with_avatar_height"
+    android:layout_height="56dp"
     android:background="@drawable/list_item_selector_background">
 
     <ImageView
         android:id="@+id/reorder"
-        android:layout_width="@dimen/material_component_lists_single_line_with_avatar_height"
-        android:layout_height="@dimen/material_component_lists_single_line_with_avatar_height"
+        android:layout_width="wrap_content"
+        android:layout_height="wrap_content"
+        android:padding="16dp"
         android:scaleType="center"
         app:srcCompat="@drawable/ic_drag_handle_24dp"
         app:tint="?android:attr/textColorHint"

+ 1 - 1
app/src/main/res/layout/common_dialog_with_checkbox.xml

@@ -15,7 +15,7 @@
         android:layout_width="match_parent"
         android:layout_height="wrap_content"
         android:layout_marginStart="-5dp"
-        android:layout_marginTop="@dimen/material_component_dialogs_padding_between_text_and_touch_target"
+        android:layout_marginTop="24dp"
         android:layout_marginEnd="0dp" />
 
 </LinearLayout>

+ 8 - 7
app/src/main/res/layout/download_item.xml

@@ -4,14 +4,15 @@
     xmlns:tools="http://schemas.android.com/tools"
     android:layout_width="match_parent"
     android:layout_height="wrap_content"
-    android:paddingTop="@dimen/material_component_lists_padding_above_list">
+    android:paddingTop="8dp">
 
     <ImageView
         android:id="@+id/reorder"
-        android:layout_width="@dimen/material_component_lists_single_line_with_avatar_height"
-        android:layout_height="0dp"
+        android:layout_width="wrap_content"
+        android:layout_height="wrap_content"
         android:layout_alignParentStart="true"
         android:layout_gravity="start"
+        android:padding="16dp"
         android:scaleType="center"
         app:layout_constraintBottom_toBottomOf="parent"
         app:layout_constraintStart_toStartOf="parent"
@@ -85,13 +86,13 @@
 
     <ImageButton
         android:id="@+id/menu"
-        android:layout_width="44dp"
-        android:layout_height="@dimen/material_component_lists_single_line_with_avatar_height"
+        android:layout_width="wrap_content"
+        android:layout_height="wrap_content"
         android:layout_toEndOf="@id/download_progress_text"
         android:background="?selectableItemBackgroundBorderless"
         android:contentDescription="@string/action_menu"
-        android:paddingStart="10dp"
-        android:paddingEnd="10dp"
+        android:paddingVertical="16dp"
+        android:paddingHorizontal="10dp"
         app:layout_constraintBottom_toBottomOf="parent"
         app:layout_constraintEnd_toEndOf="parent"
         app:layout_constraintTop_toTopOf="parent"

+ 4 - 4
app/src/main/res/layout/global_search_controller_card.xml

@@ -46,7 +46,7 @@
             android:layout_width="wrap_content"
             android:layout_height="wrap_content"
             android:contentDescription="@string/label_more"
-            android:padding="@dimen/material_component_text_fields_padding_above_and_below_label"
+            android:padding="16dp"
             app:layout_constraintBottom_toBottomOf="parent"
             app:layout_constraintEnd_toEndOf="parent"
             app:layout_constraintTop_toTopOf="parent"
@@ -59,9 +59,9 @@
         android:id="@+id/no_results_found"
         android:layout_width="match_parent"
         android:layout_height="wrap_content"
-        android:paddingStart="@dimen/material_component_text_fields_padding_above_and_below_label"
-        android:paddingEnd="@dimen/material_component_text_fields_padding_above_and_below_label"
-        android:paddingBottom="@dimen/material_component_text_fields_padding_above_and_below_label"
+        android:paddingStart="16dp"
+        android:paddingEnd="16dp"
+        android:paddingBottom="16dp"
         android:text="@string/no_results_found"
         android:visibility="gone" />
 

+ 1 - 1
app/src/main/res/layout/navigation_view_checkbox.xml

@@ -16,7 +16,7 @@
         android:clickable="false"
         android:gravity="center_vertical|start"
         android:maxLines="1"
-        android:paddingStart="@dimen/material_component_lists_icon_left_padding"
+        android:paddingHorizontal="16dp"
         android:textAppearance="@style/TextAppearance.AppCompat.Body1" />
 
 </LinearLayout>

+ 1 - 1
app/src/main/res/layout/navigation_view_checkedtext.xml

@@ -12,7 +12,7 @@
         android:layout_width="0dp"
         android:layout_height="match_parent"
         android:layout_weight="1"
-        android:drawablePadding="@dimen/material_component_lists_icon_left_padding"
+        android:drawablePadding="16dp"
         android:gravity="center_vertical|start"
         android:maxLines="1"
         android:textAppearance="@style/TextAppearance.AppCompat.Body1" />

+ 1 - 1
app/src/main/res/layout/navigation_view_radio.xml

@@ -16,7 +16,7 @@
         android:clickable="false"
         android:gravity="center_vertical|start"
         android:maxLines="1"
-        android:paddingStart="@dimen/material_component_lists_icon_left_padding"
+        android:paddingHorizontal="16dp"
         android:textAppearance="@style/TextAppearance.AppCompat.Body1" />
 
 </LinearLayout>

+ 6 - 6
app/src/main/res/layout/reader_activity.xml

@@ -88,7 +88,7 @@
                     android:layout_marginEnd="8dp"
                     android:background="@drawable/reader_seekbar_button"
                     android:contentDescription="@string/action_previous_chapter"
-                    android:padding="@dimen/material_layout_keylines_screen_edge_margin"
+                    android:padding="@dimen/screen_edge_margin"
                     app:srcCompat="@drawable/ic_skip_previous_24dp"
                     app:tint="?attr/colorOnPrimary" />
 
@@ -140,7 +140,7 @@
                     android:layout_marginStart="8dp"
                     android:background="@drawable/reader_seekbar_button"
                     android:contentDescription="@string/action_next_chapter"
-                    android:padding="@dimen/material_layout_keylines_screen_edge_margin"
+                    android:padding="@dimen/screen_edge_margin"
                     app:srcCompat="@drawable/ic_skip_next_24dp"
                     app:tint="?attr/colorOnPrimary" />
 
@@ -158,7 +158,7 @@
                     android:layout_height="match_parent"
                     android:background="?selectableItemBackgroundBorderless"
                     android:contentDescription="@string/viewer"
-                    android:padding="@dimen/material_layout_keylines_screen_edge_margin"
+                    android:padding="@dimen/screen_edge_margin"
                     app:layout_constraintEnd_toStartOf="@id/action_crop_borders"
                     app:layout_constraintStart_toStartOf="parent"
                     app:layout_constraintTop_toTopOf="parent"
@@ -171,7 +171,7 @@
                     android:layout_height="match_parent"
                     android:background="?selectableItemBackgroundBorderless"
                     android:contentDescription="@string/pref_crop_borders"
-                    android:padding="@dimen/material_layout_keylines_screen_edge_margin"
+                    android:padding="@dimen/screen_edge_margin"
                     app:layout_constraintEnd_toStartOf="@id/action_rotation"
                     app:layout_constraintStart_toEndOf="@+id/action_reading_mode"
                     app:layout_constraintTop_toTopOf="parent"
@@ -184,7 +184,7 @@
                     android:layout_height="match_parent"
                     android:background="?selectableItemBackgroundBorderless"
                     android:contentDescription="@string/pref_rotation_type"
-                    android:padding="@dimen/material_layout_keylines_screen_edge_margin"
+                    android:padding="@dimen/screen_edge_margin"
                     app:layout_constraintEnd_toStartOf="@id/action_settings"
                     app:layout_constraintStart_toEndOf="@+id/action_crop_borders"
                     app:layout_constraintTop_toTopOf="parent"
@@ -197,7 +197,7 @@
                     android:layout_height="match_parent"
                     android:background="?selectableItemBackgroundBorderless"
                     android:contentDescription="@string/action_settings"
-                    android:padding="@dimen/material_layout_keylines_screen_edge_margin"
+                    android:padding="@dimen/screen_edge_margin"
                     app:layout_constraintEnd_toEndOf="parent"
                     app:layout_constraintStart_toEndOf="@id/action_rotation"
                     app:layout_constraintTop_toTopOf="parent"

+ 1 - 1
app/src/main/res/layout/reader_color_filter_settings.xml

@@ -225,7 +225,7 @@
             android:layout_height="wrap_content"
             android:layout_marginStart="8dp"
             android:layout_marginEnd="8dp"
-            android:padding="@dimen/material_component_text_fields_floating_label_padding_between_label_and_input_text"
+            android:padding="8dp"
             app:layout_constraintEnd_toStartOf="@id/txt_brightness_seekbar_value"
             app:layout_constraintStart_toEndOf="@id/txt_brightness_seekbar_icon"
             app:layout_constraintTop_toBottomOf="@id/custom_brightness"

+ 2 - 4
app/src/main/res/layout/section_header_item.xml

@@ -9,10 +9,8 @@
         style="@style/TextAppearance.Medium.SubHeading"
         android:layout_width="match_parent"
         android:layout_height="wrap_content"
-        android:paddingStart="@dimen/material_component_text_fields_padding_above_and_below_label"
-        android:paddingTop="8dp"
-        android:paddingEnd="@dimen/material_component_text_fields_padding_above_and_below_label"
-        android:paddingBottom="8dp"
+        android:paddingHorizontal="16dp"
+        android:paddingVertical="8dp"
         tools:text="Title" />
 
 </FrameLayout>

+ 3 - 3
app/src/main/res/layout/source_list_item.xml

@@ -3,7 +3,7 @@
     xmlns:app="http://schemas.android.com/apk/res-auto"
     xmlns:tools="http://schemas.android.com/tools"
     android:layout_width="match_parent"
-    android:layout_height="@dimen/material_component_lists_single_line_with_avatar_height"
+    android:layout_height="56dp"
     android:layout_gravity="center_vertical"
     android:background="@drawable/list_item_selector_background"
     android:paddingStart="8dp"
@@ -11,8 +11,8 @@
 
     <ImageView
         android:id="@+id/thumbnail"
-        android:layout_width="@dimen/material_component_lists_single_line_with_avatar_height"
-        android:layout_height="@dimen/material_component_lists_single_line_with_avatar_height"
+        android:layout_width="56dp"
+        android:layout_height="56dp"
         android:layout_gravity="center_vertical"
         android:padding="8dp"
         android:scaleType="centerCrop"

+ 1 - 1
app/src/main/res/layout/updates_item.xml

@@ -3,7 +3,7 @@
     xmlns:app="http://schemas.android.com/apk/res-auto"
     xmlns:tools="http://schemas.android.com/tools"
     android:layout_width="match_parent"
-    android:layout_height="@dimen/material_component_lists_two_line_height"
+    android:layout_height="56dp"
     android:background="@drawable/list_item_selector_background"
     android:paddingEnd="4dp">
 

+ 4 - 0
app/src/main/res/values-sw600dp/dimens.xml

@@ -0,0 +1,4 @@
+<?xml version="1.0" encoding="utf-8"?>
+<resources>
+    <dimen name="screen_edge_margin">24dp</dimen>
+</resources>

+ 2 - 0
app/src/main/res/values/dimens.xml

@@ -12,4 +12,6 @@
     <dimen name="fab_list_padding">88dp</dimen>
 
     <dimen name="action_toolbar_list_padding">72dp</dimen>
+
+    <dimen name="screen_edge_margin">16dp</dimen>
 </resources>

+ 5 - 11
app/src/main/res/values/styles.xml

@@ -177,17 +177,11 @@
     <style name="Theme.Widget.CardView" parent="CardView">
         <item name="android:layout_width">match_parent</item>
         <item name="android:layout_height">wrap_content</item>
-        <item name="android:padding">@dimen/material_component_cards_top_and_bottom_padding</item>
-        <item name="android:layout_marginTop">@dimen/material_component_cards_space_between_cards
-        </item>
-        <item name="android:layout_marginBottom">
-            @dimen/material_component_cards_space_between_cards
-        </item>
-        <item name="android:layout_marginStart">
-            @dimen/material_component_cards_space_between_cards
-        </item>
-        <item name="android:layout_marginEnd">@dimen/material_component_cards_space_between_cards
-        </item>
+        <item name="android:padding">16dp</item>
+        <item name="android:layout_marginTop">8dp</item>
+        <item name="android:layout_marginBottom">8dp</item>
+        <item name="android:layout_marginStart">8dp</item>
+        <item name="android:layout_marginEnd">8dp</item>
         <item name="cardBackgroundColor">?attr/colorSurface</item>
         <item name="cardCornerRadius">@dimen/card_radius</item>
         <item name="cardElevation">2dp</item>