Skip to content

Conversation

@bobrippling
Copy link
Collaborator

@bobrippling bobrippling commented Nov 16, 2025

Previously if we didn't have GPS in the small snapshot we fetch:

var tries = 100;
var l = data;
while (l && l.split(",")[lIdx]==="" && tries--)
l = f.readLine();
if (l) data = l;

then we wouldn't create the map element:

${trackData.Latitude ? `
<div id="map-${track.number}" class="leaflet-map"></div>
<div class="track-stats">
<span id="stats-${track.number}">Loading...</span>
</div>` : ''}

We now create the map element and populate it with the track coords, or if no coords are found in the entire track, we hide it.

@bobrippling bobrippling force-pushed the fix/recorder-late-map branch from 6b0e56d to d4b7f23 Compare November 16, 2025 10:40
@bobrippling bobrippling force-pushed the fix/recorder-late-map branch from d4b7f23 to 180c4a9 Compare November 16, 2025 10:43
@bobrippling bobrippling marked this pull request as ready for review November 16, 2025 10:46
@bobrippling
Copy link
Collaborator Author

No problems with testing - merging

@bobrippling bobrippling merged commit 9a0f037 into espruino:master Nov 19, 2025
1 check passed
@bobrippling bobrippling deleted the fix/recorder-late-map branch November 19, 2025 08:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant