Skip to content

Save visibility of wpt groups#1522

Open
Dima-1 wants to merge 20 commits intomainfrom
hide-wpt-group
Open

Save visibility of wpt groups#1522
Dima-1 wants to merge 20 commits intomainfrom
hide-wpt-group

Conversation

@Dima-1
Copy link
Contributor

@Dima-1 Dima-1 commented Mar 12, 2026

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds persistence for waypoint-group visibility so waypoint visibility does not reset after refresh/logout (Issue #712) by saving group hidden flags and reapplying them when tracks are rendered.

Changes:

  • Introduces INFO_FILE_EXT constant and uses it where .info filenames are constructed/filtered.
  • Saves waypoint-group visibility (pointsGroups.*.ext.hidden) for cloud tracks by uploading an updated .info file.
  • Applies waypoint-group visibility when redrawing waypoint markers on existing Leaflet layers.

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 11 comments.

Show a summary per file
File Description
map/src/menu/settings/CloudSettings.jsx Filters out .info files using a shared extension constant.
map/src/map/util/TrackLayerProvider.js Stores pointsGroups on the layer and uses it to hide/show waypoint markers during redraw.
map/src/manager/track/TracksManager.js Exports INFO_FILE_EXT and uses it when building .info download URLs.
map/src/manager/track/SaveTrackManager.js Adds saveInfoFile to upload .info updates containing waypoint-group visibility.
map/src/infoblock/components/tabs/WaypointsTab.jsx Updates waypoint-group visibility state and debounced saving for cloud tracks.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@Dima-1 Dima-1 marked this pull request as ready for review March 18, 2026 16:21
Copy link
Contributor

@alisa911 alisa911 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Add a test and test.

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.

3 participants