From c0a98e4d491714bb50147152a002a413585b3c54 Mon Sep 17 00:00:00 2001 From: Doychin Atanasov Date: Thu, 30 Jan 2014 14:34:56 +0200 Subject: [PATCH] Playlist add-on: no repetitive searching in the DOM in _refresh --- add-on/jplayer.playlist.js | 15 ++++++++------- 1 file changed, 8 insertions(+), 7 deletions(-) diff --git a/add-on/jplayer.playlist.js b/add-on/jplayer.playlist.js index 60d48736..921dca15 100644 --- a/add-on/jplayer.playlist.js +++ b/add-on/jplayer.playlist.js @@ -198,22 +198,23 @@ }, _refresh: function(instant) { /* instant: Can be undefined, true or a function. - * undefined -> use animation timings - * true -> no animation - * function -> use animation timings and excute function at half way point. + * undefined -> use animation timings + * true -> no animation + * function -> use animation timings and excute function at half way point. */ var self = this; + var playlist_ul = $(self.cssSelector.playlist + " ul"); if(instant && !$.isFunction(instant)) { $(this.cssSelector.playlist + " ul").empty(); + $.each(this.playlist, function(i) { - $(self.cssSelector.playlist + " ul").append(self._createListItem(self.playlist[i])); + playlist_ul.append(self._createListItem(self.playlist[i])); }); this._updateControls(); } else { - var displayTime = $(this.cssSelector.playlist + " ul").children().length ? this.options.playlistOptions.displayTime : 0; - - $(this.cssSelector.playlist + " ul").slideUp(displayTime, function() { + var displayTime = playlist_ul.children().length ? this.options.playlistOptions.displayTime : 0; + playlist_ul.slideUp(displayTime, function() { var $this = $(this); $(this).empty();