Skip to content

remove stale layer style data during deletion and load properly - #451

Merged
BryonLewis merged 2 commits into
masterfrom
delete-style-persistence
Aug 19, 2026
Merged

remove stale layer style data during deletion and load properly#451
BryonLewis merged 2 commits into
masterfrom
delete-style-persistence

Conversation

@BryonLewis

Copy link
Copy Markdown
Collaborator

resovles #449

The style editor was using stale data from selectedLayerStyles in Pinia sotre and layer.default_style on the selected layer object

Delete now uses a new function applyStyleSelection which will set the style or a default None style if the style is undefined

resetCurrentStyle in LayerStyle.vue has been updated so that it will no longer rely blindly on currentLayerStyle.value?.id but will check availableStyles for the style and make sure it is still valid. If it is not still valid it will select the next style that is labeled is_default and finally if nothing is loaded or found it will use the none style.

in fetchAvailableLayer in layer.ts before it would only update the availableLayers it has changed to updated the selectedLayers to keep them in sync with the availableLayers before they could get out of sync.

@BryonLewis BryonLewis changed the title fix: remove stale layer style data during deletion and load properly remove stale layer style data during deletion and load properly Aug 17, 2026
@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Aug 17, 2026

Copy link
Copy Markdown

Deploying geodatalytics with  Cloudflare Pages  Cloudflare Pages

Latest commit: 17afb44
Status: ✅  Deploy successful!
Preview URL: https://ca4b760b.geodatalytics.pages.dev
Branch Preview URL: https://delete-style-persistence.geodatalytics.pages.dev

View logs

Comment thread web/src/components/sidebars/LayerStyle.vue Outdated
@BryonLewis
BryonLewis merged commit 6f0b3aa into master Aug 19, 2026
4 checks passed
@BryonLewis
BryonLewis deleted the delete-style-persistence branch August 19, 2026 16:39
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.

2 participants