Skip to content

Commit c93c7a5

Browse files
committed
Minor UI fixes
1 parent 06d68a8 commit c93c7a5

File tree

6 files changed

+40
-7
lines changed

6 files changed

+40
-7
lines changed

app/src/main/java/biz/dealnote/messenger/player/MusicPlaybackService.java

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -668,7 +668,6 @@ private void fetchCoverAndUpdateMetadata() {
668668

669669
PicassoInstance.with()
670670
.load(getAlbumCover())
671-
.config(Bitmap.Config.RGB_565)
672671
.into(new Target() {
673672
@Override
674673
public void onBitmapLoaded(Bitmap bitmap, Picasso.LoadedFrom from) {

app/src/main/java/biz/dealnote/messenger/player/NotificationHelper.java

Lines changed: 8 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -74,13 +74,13 @@ public void buildNotification(Context context, final String artistName,
7474
.setShowCancelButton(true)
7575
.setShowActionsInCompactView(0, 1, 2)
7676
.setCancelButtonIntent(retreivePlaybackActions(4)))
77-
.addAction(new NotificationCompat.Action(R.drawable.page_first,
77+
.addAction(new NotificationCompat.Action(R.drawable.prev_notification,
7878
context.getResources().getString(R.string.previous),
7979
retreivePlaybackActions(ACTION_PREV)))
80-
.addAction(new NotificationCompat.Action(isPlaying ? R.drawable.pause : R.drawable.play,
80+
.addAction(new NotificationCompat.Action(isPlaying ? R.drawable.pause_notification : R.drawable.play_notification,
8181
context.getResources().getString(isPlaying ? R.string.pause : R.string.play),
8282
retreivePlaybackActions(ACTION_PLAY_PAUSE)))
83-
.addAction(new NotificationCompat.Action(R.drawable.page_last,
83+
.addAction(new NotificationCompat.Action(R.drawable.next_notification,
8484
context.getResources().getString(R.string.next),
8585
retreivePlaybackActions(ACTION_NEXT)));
8686

@@ -100,7 +100,6 @@ public void killNotification() {
100100
public void updateNotificationCover(String cover) {
101101
PicassoInstance.with()
102102
.load(cover)
103-
.config(Bitmap.Config.RGB_565)
104103
.into(new Target() {
105104
@Override
106105
public void onBitmapLoaded(Bitmap bitmap, Picasso.LoadedFrom from) {
@@ -133,8 +132,11 @@ public void updatePlayState(final boolean isPlaying) {
133132
}
134133
//Remove pause action
135134
mNotificationBuilder.mActions.remove(1);
136-
mNotificationBuilder.mActions.add(1, new android.support.v4.app.NotificationCompat.Action(isPlaying ? R.drawable.pause : R.drawable.play, ""
137-
, retreivePlaybackActions(1)));
135+
mNotificationBuilder.mActions.add(1,
136+
new android.support.v4.app.NotificationCompat.Action(
137+
isPlaying ? R.drawable.pause_notification : R.drawable.play_notification,
138+
null,
139+
retreivePlaybackActions(1)));
138140

139141
mNotificationManager.notify(PHOENIX_MUSIC_SERVICE, mNotificationBuilder.build());
140142
}
Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,8 @@
1+
<!-- drawable/page_last.xml -->
2+
<vector xmlns:android="http://schemas.android.com/apk/res/android"
3+
android:height="24dp"
4+
android:width="24dp"
5+
android:viewportWidth="24"
6+
android:viewportHeight="24">
7+
<path android:fillColor="#fff" android:pathData="M5.59,7.41L10.18,12L5.59,16.59L7,18L13,12L7,6L5.59,7.41M16,6H18V18H16V6Z" />
8+
</vector>
Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,8 @@
1+
<!-- drawable/pause.xml -->
2+
<vector xmlns:android="http://schemas.android.com/apk/res/android"
3+
android:height="40dp"
4+
android:width="40dp"
5+
android:viewportWidth="24"
6+
android:viewportHeight="24">
7+
<path android:fillColor="#fff" android:pathData="M14,19H18V5H14M6,19H10V5H6V19Z" />
8+
</vector>
Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,8 @@
1+
<!-- drawable/play.xml -->
2+
<vector xmlns:android="http://schemas.android.com/apk/res/android"
3+
android:height="40dp"
4+
android:width="40dp"
5+
android:viewportWidth="24"
6+
android:viewportHeight="24">
7+
<path android:fillColor="#fff" android:pathData="M8,5.14V19.14L19,12.14L8,5.14Z" />
8+
</vector>
Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,8 @@
1+
<!-- drawable/page_first.xml -->
2+
<vector xmlns:android="http://schemas.android.com/apk/res/android"
3+
android:height="24dp"
4+
android:width="24dp"
5+
android:viewportWidth="24"
6+
android:viewportHeight="24">
7+
<path android:fillColor="#fff" android:pathData="M18.41,16.59L13.82,12L18.41,7.41L17,6L11,12L17,18L18.41,16.59M6,6H8V18H6V6Z" />
8+
</vector>

0 commit comments

Comments
 (0)