Skip to content
This repository has been archived by the owner on Nov 3, 2021. It is now read-only.

Commit

Permalink
Bug 841977 - Part 2: Show track number instead of index. Also disc nu…
Browse files Browse the repository at this point in the history
…mber if relevant.
  • Loading branch information
hfiguiere committed Sep 24, 2014
1 parent ae18fb7 commit 97b0ec1
Showing 1 changed file with 11 additions and 1 deletion.
12 changes: 11 additions & 1 deletion apps/music/js/music.js
Expand Up @@ -1180,7 +1180,12 @@ function createListElement(option, data, index, highlight) {

var indexSpan = document.createElement('span');
indexSpan.className = 'list-song-index';
indexSpan.textContent = index + 1;
var trackNum = data.metadata.tracknum;
if (data.multidisc) {
trackNum = data.metadata.discnum + '.' +
(trackNum < 10 ? '0' + trackNum : trackNum);
}
indexSpan.textContent = trackNum;

var titleSpan = document.createElement('span');
titleSpan.className = 'list-song-title';
Expand Down Expand Up @@ -1746,9 +1751,13 @@ var SubListView = {
function lv_enumerateAll(dataArray) {
var albumName;
var albumNameL10nId;
var maxDiscNum = 1;

if (option === 'album') {
dataArray.sort(function(e1, e2) {
maxDiscNum = Math.max(maxDiscNum, e1.metadata.disccount,
e2.metadata.disccount,
e1.metadata.discnum, e2.metadata.discnum);
return (e1.metadata.discnum - e2.metadata.discnum) ||
(e1.metadata.tracknum - e2.metadata.tracknum);
});
Expand All @@ -1774,6 +1783,7 @@ var SubListView = {
SubListView.setAlbumSrc(data);

dataArray.forEach(function(songData) {
songData.multidisc = (maxDiscNum > 1);
SubListView.update(songData);
});

Expand Down

0 comments on commit 97b0ec1

Please sign in to comment.