Inconsistency in counting loaded tracks/files

[hajduakos]

When dragging and dropping files, I observed the loading screen to say "240 loaded of 238". I think this inconsistency comes from the fact that the total is given by the number of files (numFiles):

derive/src/ui.js

Line 130 in ed9d902

<p>${numLoaded} loaded${failureString} of <b>${numFiles}</b></p>`;

but numLoaded is incremented for each track (and not for each file):

derive/src/ui.js

Lines 89 to 96 in ed9d902

	const handleTrackFile = async (file) => {
	for (const track of await extractTracks(file)) {
	track.filename = file.name;
	tracks.push(track);
	map.addTrack(track);
	modal.addSuccess();
	}
	};

So if there are files with multiple tracks, this inconsistency can occur that the number of loaded is more than the number of total.

Perhaps a quick fix would be to move modal.addSuccess(); out of the loop?

[erik]

Nice catch! Thanks for digging into the cause as well.