From 152e7ed89a748810fbba04906cbedb735ddc443e Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Pauli=20J=C3=A4rvinen?= Date: Sun, 18 Jun 2023 22:04:26 +0300 Subject: [PATCH] Files playlist tab: Show the file path or stream URL on tooltip --- CHANGELOG.md | 1 + js/embedded/playlisttabview.js | 7 ++++++- 2 files changed, 7 insertions(+), 1 deletion(-) diff --git a/CHANGELOG.md b/CHANGELOG.md index 1ac930b67..50eb96f83 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -1,6 +1,7 @@ ## [Unreleased] ### Added +- Files playlist tab: Tooltip showing the file path or stream URL ### Changed - Drop the logic migrating user settings from cookies to localStorage diff --git a/js/embedded/playlisttabview.js b/js/embedded/playlisttabview.js index 90004c891..7d9301cc3 100644 --- a/js/embedded/playlisttabview.js +++ b/js/embedded/playlisttabview.js @@ -45,10 +45,15 @@ OCA.Music.initPlaylistTabView = function(playlistMimes) { return file.caption || OCA.Music.Utils.titleFromFilename(file.name); }; + let tooltipForFile = function(file) { + return file.url || `${file.path}/${file.name}`; + }; + for (let i = 0; i < data.files.length; ++i) { list.append($(document.createElement('li')) .attr('id', 'music-playlist-item-' + i) - .text(titleForFile(data.files[i]))); + .text(titleForFile(data.files[i])) + .prop('title', tooltipForFile(data.files[i]))); } // click handler