Skip to content

Map Editor in Portals - Part 4 - Editing UI - #2948

Draft
robyngit wants to merge 9 commits into
feature-1995-map-editor-part3from
feature-1995-map-editor-part4
Draft

robyngit wants to merge 9 commits into
feature-1995-map-editor-part3from
feature-1995-map-editor-part4

Conversation

@robyngit

@robyngit robyngit commented Oct 7, 2026

Copy link
Copy Markdown
Member

This PR adds the UI for editing layers in Cesium maps.

Include tests for this validation

Issue #1995
- Add layer selection and settings panels, including hidden and categorized layers
- Keep WMTS, Ion, and OpenStreetMap edits when switching layers. Validate them on portal Save, show inline errors, and focus invalid fields
- Export current imagery settings and preserve NaturalEarthII and USGSImageryTopo aliases. Expand them when creating Cesium providers
- Clean up map editors and listeners when rerendering or deleting sections
- Test editing, validation, cleanup, provider rendering

Issue #1995
- Add remove button with hover and keyboard focus removal previews

Issue #1995
- Add "Add layer" button to each layer group in the map editor.
- Focus the label of a newly added layer for immediate editing.
- Remove the "preview remove" behaviour. Making the layers transparent reduced contrast below the acceptable level.

Issue #1995
- Make a view to handle the list of map assets and their editing forms, and a view to handle individual map asset rows
- Move logic out of the Map Editor into the new views
- Indicate a button will remove a row with strike through styling

Issue #1995
- Reuse layer styles from live map with accessible, collapsible categories
- Scroll the list and settings form independently
- Simplify add and remove controls and remove redundant guidance

Issue #1995
@robyngit
robyngit added this pull request to stack #2895 October 7, 2026 23:34
- Move category UI into the new MapAssetCategoryView
- Prevent re-rendering all rows when collection changes (in preparation for layer reordering)

Issue #1995

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

1 participant