Always show poster images for audio playlists

src/playlist-maker.js

@@ -319,7 +319,7 @@ export default function factory(player, initialList, initialIndex = 0) {
       // this by hiding every poster after the first play list item. This
       // doesn't cover every use case for showing/hiding the poster, but
       // it will significantly improve the user experience.
-      if (index > 0) {
+      if (index > 0 && !player.isAudio()) {
         player.poster('');
       }
 

test/player-proxy-maker.js

@@ -3,6 +3,7 @@ import videojs from 'video.js';
 
 const proxy = (props) => {
   let poster_ = '';
+  let audio_ = false;
   const player = Object.assign({}, videojs.EventTarget.prototype, {
     play: () => {},
     paused: () => {},
@@ -15,6 +16,11 @@ const proxy = (props) => {
     src: () => {},
     currentSrc: () => {},
     addRemoteTextTrack: () => {},
+    isAudio: (audio) => {
+      if (audio !== undefined) {
+        audio_ = audio;
+      } return audio_;
+    },
     removeRemoteTextTrack: () => {},
     remoteTextTracks: () => {},
     playlist: () => [],

test/playlist-maker.test.js

@@ -185,6 +185,17 @@ QUnit.test('playlist.currentItem() hides the poster for all videos after the fir
   }
 });
 
+QUnit.test('playlist.currentItem() always shows the poster for audio players', function(assert) {
+  const player = playerProxyMaker();
+  const playlist = playlistMaker(player, videoList);
+
+  player.isAudio(true);
+  for (let i = 1; i <= playlist.lastIndex(); i++) {
+    playlist.currentItem(i);
+    assert.notEqual(player.poster(), '', 'poster is shown for audio playlist index ' + i);
+  }
+});
+
 QUnit.test('playlist.currentItem() returns -1 with an empty playlist', function(assert) {
   const playlist = playlistMaker(playerProxyMaker(), []);