Skip to content

Performance: Cleanup/Refactor#26

Merged
hgosalia merged 2 commits into
mainfrom
perf/cleanup
Jun 5, 2026
Merged

Performance: Cleanup/Refactor#26
hgosalia merged 2 commits into
mainfrom
perf/cleanup

Conversation

@hgosalia

@hgosalia hgosalia commented Jun 5, 2026

Copy link
Copy Markdown
Member

No description provided.

@github-actions

github-actions Bot commented Jun 5, 2026

Copy link
Copy Markdown

📋 PR Summary

Title: Performance: Cleanup/Refactor
11 files changed: +161 additions, -267 deletions

Description

This PR introduces several performance-related cleanups and refactors across the application. Notably, it removes one-time migration functions for image formats from the init function in data.js, streamlining the initialization process since these migrations are no longer necessary. Additionally, it consolidates the logic for building playback stops into a new buildPlaybackStops function, enhancing code readability and maintainability while ensuring that playback functionality remains intact. The PR also implements a throttling mechanism for Nominatim API calls, improving the efficiency of geocoding operations by centralizing the rate-limiting logic into a dedicated function, nominatimThrottle.

Files

  • README.md — Updated (+4/-4)
  • css/styles.css — Updated .logo-text, .export-overlay-title
  • js/data.js — Modified: init
  • js/export.js — Modified: exportVideo
  • js/map.js — New: _syncExportBtnState, _initMapOverlays, _initMapControls. Modified: initTheme, _doStyleSwap, initMap, setMapStyle
  • js/media.js — Modified: processFiles
  • js/modals.js — Modified: geocodePlace
  • js/pins.js — Modified: reverseGeocode
  • js/playback.js — Modified: togglePlayback
  • js/search.js — Modified: renderSearchResults, runCategorySearch, runDestSearch
  • js/state.js — New: nominatimThrottle, buildPlaybackStops

@hgosalia hgosalia merged commit 16e51a2 into main Jun 5, 2026
1 check passed
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