From 45f6795ccfcfa270fd58d22fd23411dd088d0eb7 Mon Sep 17 00:00:00 2001 From: casey langen Date: Sat, 25 Mar 2017 23:06:42 -0700 Subject: [PATCH] Updated CHANGELOG and fixed a couple compiler warnings. --- CHANGELOG.txt | 1 + src/musikbox/app/overlay/PlayQueueOverlays.cpp | 4 ++-- 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/CHANGELOG.txt b/CHANGELOG.txt index b06cf0326..64f54ac9e 100644 --- a/CHANGELOG.txt +++ b/CHANGELOG.txt @@ -6,6 +6,7 @@ user-facing: bring up a context menu with the ability to jump to the album or enqueue the album's tracks for playback. this also had a nice side effect of making scrolling more fluid! +* added jumping to album/artist/genre to the selected track context menu * fixed a really old bug that could cause lists to automatically scroll to weird places if the window or view is resized * added localization support for "dimensions". this allows for per-locale sizing diff --git a/src/musikbox/app/overlay/PlayQueueOverlays.cpp b/src/musikbox/app/overlay/PlayQueueOverlays.cpp index da32431b0..50f4a0c3e 100644 --- a/src/musikbox/app/overlay/PlayQueueOverlays.cpp +++ b/src/musikbox/app/overlay/PlayQueueOverlays.cpp @@ -297,7 +297,7 @@ void PlayQueueOverlays::ShowAddTrackOverlay( [track, &messageQueue, &playback] (ListOverlay* overlay, IScrollAdapterPtr adapter, size_t index) { auto editor = playback.Edit(); - if (index >= 0 && index <= 2) { + if (index <= 2) { handleJumpTo(index, messageQueue, track); } else if (index == 3) { /* end */ @@ -375,7 +375,7 @@ void PlayQueueOverlays::ShowAlbumDividerOverlay( } /* items 0, 1, and 2 jump to category */ - if (index >= 0 && index <= 2) { + if (index <= 2) { handleJumpTo(index, messageQueue, firstTrack); } /* the other are our standard play queue operations */