Skip to content

Commit df1681b

Browse files
committed
Redesigned playlist control layout: icons above labels with better spacing and colour
1 parent 45589db commit df1681b

1 file changed

Lines changed: 46 additions & 27 deletions

File tree

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

Lines changed: 46 additions & 27 deletions
Original file line numberDiff line numberDiff line change
@@ -8,23 +8,29 @@
88

99
<LinearLayout
1010
android:id="@+id/playlist_ctrl_play_bg_button"
11-
android:layout_width="match_parent"
12-
android:layout_height="@dimen/playlist_ctrl_height"
11+
android:layout_width="0dp"
12+
android:layout_height="wrap_content"
1313
android:layout_weight="1"
14+
android:orientation="vertical"
15+
android:gravity="center"
1416
android:background="?attr/selectableItemBackground"
1517
android:clickable="true"
1618
android:focusable="true"
17-
android:gravity="center">
19+
android:paddingVertical="8dp">
1820

19-
<org.schabi.newpipe.views.NewPipeTextView
21+
<ImageView
22+
android:layout_width="24dp"
23+
android:layout_height="24dp"
24+
android:src="@drawable/ic_headset"
25+
android:contentDescription="@string/controls_background_title" />
26+
27+
<TextView
2028
android:layout_width="wrap_content"
2129
android:layout_height="wrap_content"
22-
android:drawableStart="@drawable/ic_headset"
23-
android:drawablePadding="4dp"
24-
android:gravity="center_vertical"
2530
android:text="@string/controls_background_title"
26-
android:textColor="?attr/colorAccent"
27-
android:textSize="@dimen/channel_rss_title_size" />
31+
android:textColor="#6C6C6C"
32+
android:textSize="@dimen/channel_rss_title_size"
33+
android:gravity="center" />
2834
</LinearLayout>
2935

3036
<View
@@ -37,23 +43,29 @@
3743

3844
<LinearLayout
3945
android:id="@+id/playlist_ctrl_play_all_button"
40-
android:layout_width="match_parent"
41-
android:layout_height="match_parent"
46+
android:layout_width="0dp"
47+
android:layout_height="wrap_content"
4248
android:layout_weight="1"
49+
android:orientation="vertical"
50+
android:gravity="center"
4351
android:background="?attr/selectableItemBackground"
4452
android:clickable="true"
4553
android:focusable="true"
46-
android:gravity="center">
54+
android:paddingVertical="8dp">
55+
56+
<ImageView
57+
android:layout_width="24dp"
58+
android:layout_height="24dp"
59+
android:src="@drawable/ic_playlist_play"
60+
android:contentDescription="@string/play_all" />
4761

48-
<org.schabi.newpipe.views.NewPipeTextView
62+
<TextView
4963
android:layout_width="wrap_content"
5064
android:layout_height="wrap_content"
51-
android:drawableStart="@drawable/ic_playlist_play"
52-
android:drawablePadding="4dp"
53-
android:gravity="center_vertical"
5465
android:text="@string/play_all"
55-
android:textColor="?attr/colorAccent"
56-
android:textSize="@dimen/channel_rss_title_size" />
66+
android:textColor="#6C6C6C"
67+
android:textSize="@dimen/channel_rss_title_size"
68+
android:gravity="center" />
5769
</LinearLayout>
5870

5971
<View
@@ -66,22 +78,29 @@
6678

6779
<LinearLayout
6880
android:id="@+id/playlist_ctrl_play_popup_button"
69-
android:layout_width="match_parent"
70-
android:layout_height="match_parent"
81+
android:layout_width="0dp"
82+
android:layout_height="wrap_content"
7183
android:layout_weight="1"
84+
android:orientation="vertical"
85+
android:gravity="center"
7286
android:background="?attr/selectableItemBackground"
7387
android:clickable="true"
7488
android:focusable="true"
75-
android:gravity="center">
89+
android:paddingVertical="8dp">
90+
91+
<ImageView
92+
android:layout_width="24dp"
93+
android:layout_height="24dp"
94+
android:src="@drawable/ic_picture_in_picture"
95+
android:contentDescription="@string/controls_popup_title" />
7696

77-
<org.schabi.newpipe.views.NewPipeTextView
97+
<TextView
7898
android:layout_width="wrap_content"
7999
android:layout_height="wrap_content"
80-
android:drawableStart="@drawable/ic_picture_in_picture"
81-
android:drawablePadding="4dp"
82-
android:gravity="center_vertical"
83100
android:text="@string/controls_popup_title"
84-
android:textColor="?attr/colorAccent"
85-
android:textSize="@dimen/channel_rss_title_size" />
101+
android:textColor="#6C6C6C"
102+
android:textSize="@dimen/channel_rss_title_size"
103+
android:gravity="center" />
86104
</LinearLayout>
105+
87106
</LinearLayout>

0 commit comments

Comments
 (0)