Skip to content

Commit 993dcd9

Browse files
authored
Fix: Some white theme issues with dialog text and quality profiles
1 parent bcd9d65 commit 993dcd9

File tree

4 files changed

+42
-26
lines changed

4 files changed

+42
-26
lines changed

app/src/main/res/layout/account_edit_dialog.xml

Lines changed: 23 additions & 21 deletions
Original file line numberDiff line numberDiff line change
@@ -40,28 +40,28 @@
4040
<androidx.cardview.widget.CardView
4141
android:layout_width="@dimen/account_select_linear_item_size"
4242
android:layout_height="@dimen/account_select_linear_item_size"
43-
app:cardCornerRadius="@dimen/rounded_image_radius"
4443
android:layout_gravity="center"
45-
>
44+
app:cardCornerRadius="@dimen/rounded_image_radius">
45+
4646
<ImageView
4747
android:id="@+id/account_image"
48-
android:src="@drawable/profile_bg_blue"
49-
android:focusable="true"
50-
android:contentDescription="@string/preview_background_img_des"
51-
android:scaleType="centerCrop"
52-
android:foreground="@drawable/outline_drawable_forced_round"
5348
android:layout_width="match_parent"
5449
android:layout_height="match_parent"
55-
android:layout_gravity="center" />
50+
android:layout_gravity="center"
51+
android:contentDescription="@string/preview_background_img_des"
52+
android:focusable="true"
53+
android:foreground="@drawable/outline_drawable_forced_round"
54+
android:scaleType="centerCrop"
55+
android:src="@drawable/profile_bg_blue" />
56+
5657
<ImageView
5758
android:id="@+id/edit_profile_photo_button"
5859
android:layout_width="wrap_content"
5960
android:layout_height="wrap_content"
60-
android:src="@drawable/ic_baseline_edit_24"
6161
android:layout_gravity="bottom|end"
62-
android:padding="5dp"
6362
android:background="@color/skipOpTransparent"
64-
/>
63+
android:padding="5dp"
64+
android:src="@drawable/ic_baseline_edit_24" />
6565
</androidx.cardview.widget.CardView>
6666

6767
<EditText
@@ -82,47 +82,49 @@
8282
android:id="@+id/lockProfileCheckbox"
8383
android:layout_width="wrap_content"
8484
android:layout_height="wrap_content"
85-
android:text="@string/lock_profile" />
85+
android:buttonTint="?attr/textColor"
86+
android:text="@string/lock_profile"
87+
android:textColor="?attr/grayTextColor" />
8688

8789

8890
</LinearLayout>
8991

9092
<RelativeLayout
9193
android:id="@+id/apply_btt_holder"
92-
android:orientation="horizontal"
93-
android:padding="10dp"
9494
android:layout_width="match_parent"
9595
android:layout_height="60dp"
9696
android:layout_gravity="bottom"
97-
android:layout_marginTop="-60dp">
97+
android:layout_marginTop="-60dp"
98+
android:orientation="horizontal"
99+
android:padding="10dp">
98100

99101
<com.google.android.material.button.MaterialButton
100102
android:id="@+id/delete_btt"
101-
android:text="@string/delete"
103+
style="@style/BlackButton"
102104
android:layout_width="wrap_content"
103105
android:layout_alignParentStart="true"
104106
android:layout_gravity="center_vertical"
105107
android:nextFocusRight="@id/apply_btt"
106-
style="@style/BlackButton" />
108+
android:text="@string/delete" />
107109

108110
<com.google.android.material.button.MaterialButton
109111
android:id="@+id/apply_btt"
110-
android:text="@string/sort_apply"
112+
style="@style/WhiteButton"
111113
android:layout_width="wrap_content"
112114
android:layout_gravity="center_vertical|end"
113115
android:layout_toStartOf="@+id/cancel_btt"
114116
android:nextFocusLeft="@id/delete_btt"
115117
android:nextFocusRight="@id/cancel_btt"
116-
style="@style/WhiteButton" />
118+
android:text="@string/sort_apply" />
117119

118120
<com.google.android.material.button.MaterialButton
119121
android:id="@+id/cancel_btt"
120-
android:text="@string/sort_cancel"
122+
style="@style/BlackButton"
121123
android:layout_width="wrap_content"
122124
android:layout_alignParentEnd="true"
123125
android:layout_gravity="center_vertical|end"
124126
android:nextFocusLeft="@id/apply_btt"
125-
style="@style/BlackButton" />
127+
android:text="@string/sort_cancel" />
126128

127129
</RelativeLayout>
128130
</LinearLayout>

app/src/main/res/layout/confirm_exit_dialog.xml

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,9 +5,11 @@
55
android:orientation="vertical"
66
android:paddingHorizontal="16dp"
77
android:paddingVertical="8dp">
8+
89
<CheckBox
910
android:id="@+id/checkboxDontShowAgain"
1011
android:layout_width="wrap_content"
1112
android:layout_height="wrap_content"
12-
android:text="@string/dont_show_again" />
13+
android:text="@string/dont_show_again"
14+
android:textColor="?attr/grayTextColor" />
1315
</LinearLayout>

app/src/main/res/layout/player_quality_profile_item.xml

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -10,14 +10,14 @@
1010
android:id="@+id/card_view"
1111
android:layout_width="0dp"
1212
android:layout_height="0dp"
13-
android:focusable="true"
14-
app:layout_constraintDimensionRatio="1"
1513
android:layout_marginStart="10dp"
1614
android:animateLayoutChanges="true"
17-
android:backgroundTint="?attr/primaryGrayBackground"
15+
android:backgroundTint="@color/primaryGrayBackground"
16+
android:focusable="true"
1817
android:foreground="?attr/selectableItemBackgroundBorderless"
1918
app:cardCornerRadius="@dimen/rounded_image_radius"
2019
app:layout_constraintBottom_toBottomOf="parent"
20+
app:layout_constraintDimensionRatio="1"
2121
app:layout_constraintEnd_toEndOf="parent"
2222
app:layout_constraintHeight_percent="0.4"
2323
app:layout_constraintStart_toStartOf="parent"
@@ -29,7 +29,8 @@
2929
android:layout_height="match_parent"
3030
android:alpha="0.4"
3131
android:contentDescription="@string/profile_background_des"
32-
android:scaleType="centerCrop" />
32+
android:scaleType="centerCrop"
33+
tools:src="@drawable/profile_bg_teal" />
3334

3435
<View
3536
android:id="@+id/outline"
@@ -46,6 +47,7 @@
4647
android:layout_gravity="center"
4748
android:gravity="center"
4849
android:padding="10dp"
50+
android:textColor="@color/textColor"
4951
android:textSize="16sp"
5052
tools:text="@string/mobile_data" />
5153

app/src/main/res/values/styles.xml

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -478,6 +478,7 @@
478478
<item name="android:windowMinWidthMinor">@dimen/abc_dialog_min_width_minor</item>
479479
<item name="android:windowBackground">@drawable/dialog__window_background</item>
480480
<item name="textAllCaps">false</item>
481+
<item name="android:textColorPrimary">?attr/grayTextColor</item>
481482

482483
<item name="buttonBarPositiveButtonStyle">@style/WhiteButton.Dialog</item>
483484
<item name="buttonBarNegativeButtonStyle">@style/BlackButton.Dialog</item>
@@ -1034,49 +1035,58 @@
10341035
@drawable/cast_ic_mini_controller_closed_caption
10351036
</item>
10361037
</style>
1038+
10371039
<style name="RoundedImageView">
10381040
<item name="cornerFamily">rounded</item>
10391041
<item name="cornerSize">50%</item>
10401042
</style>
1043+
10411044
<style name="PlayUnderlayProgress">
10421045
<item name="shapeAppearanceOverlay">@style/RoundedImageView</item>
10431046
<item name="android:scaleType">center</item>
10441047
<item name="android:src">@drawable/netflix_play</item>
10451048
<item name="android:layout_gravity">center</item>
10461049
</style>
1050+
10471051
<style name="CircularProgress">
10481052
<item name="trackThickness">@dimen/circular_progress_thickness</item>
10491053
<item name="android:layout_gravity">center</item>
10501054
<item name="indicatorColor">@color/white</item>
10511055
</style>
1056+
10521057
<style name="HomePlayUnderlayProgress" parent="PlayUnderlayProgress">
10531058
<item name="tint">@color/white</item>
10541059
<item name="android:layout_width">@dimen/home_poster_progress_size</item>
10551060
<item name="android:layout_height">@dimen/home_poster_progress_size</item>
10561061
<item name="android:background">@color/playIconBackground</item>
10571062
</style>
1063+
10581064
<style name="HomeCircularProgress" parent="CircularProgress">
10591065
<item name="android:layout_width">@dimen/home_poster_progress_size</item>
10601066
<item name="android:layout_height">@dimen/home_poster_progress_size</item>
10611067
<item name="indicatorSize">@dimen/home_poster_progress_size</item>
10621068
</style>
1069+
10631070
<style name="EpisodePlayUnderlayProgress" parent="PlayUnderlayProgress">
10641071
<item name="tint">@color/white</item>
10651072
<item name="android:layout_width">40dp</item>
10661073
<item name="android:layout_height">40dp</item>
10671074
<item name="android:background">@color/playIconBackground</item>
10681075
</style>
1076+
10691077
<style name="EpisodeCircularProgress" parent="CircularProgress">
10701078
<item name="android:layout_width">@dimen/episode_progress_size</item>
10711079
<item name="android:layout_height">@dimen/episode_progress_size</item>
10721080
<item name="indicatorSize">@dimen/episode_progress_size</item>
10731081
</style>
1082+
10741083
<style name="ContinueWatchingPlayUnderlayProgress">
10751084
<item name="android:layout_width">@dimen/continue_watching_progress_size</item>
10761085
<item name="android:layout_height">@dimen/continue_watching_progress_size</item>
10771086
<item name="android:src">@drawable/play_button_transparent</item>
10781087
<item name="android:layout_gravity">center</item>
10791088
</style>
1089+
10801090
<style name="ContinueWatchingCircularProgress" parent="CircularProgress">
10811091
<item name="android:layout_width">@dimen/continue_watching_progress_size</item>
10821092
<item name="android:layout_height">@dimen/continue_watching_progress_size</item>

0 commit comments

Comments
 (0)