-
Notifications
You must be signed in to change notification settings - Fork 26
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
remove mapp.layer changeend method #1601
remove mapp.layer changeend method #1601
Conversation
does it affect deleting features from editable layers? when I remove a feature from a layer the deletion is complete but the feature stays on the map until reload. |
Yes, it should be possible to delete a geometry entry or delete a location with associated geometry. Screencast.from.2024-10-25.14-45-47.webm |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
All happy with this!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There is a bug with editing a location on an mvt
layer.
Steps to replicate:
- MVT Layer with a categorized thematic on field_a
- Select a location on this layer, and update the value of field_a to a different option in the thematic.
- The locations styling is not updated correctly on the mapview, the theming is not updated.
- If you zoom out or in, the thematic will update, but when you go back to the original zoom, it will be as it was initially
@simon-leech I create a separate issue for this. #1624 |
Quality Gate passedIssues Measures |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tried some editing, deleting and adding. It all looks good to me
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Re-review in progress
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
All looks good to me - tested on a range of different layer types, updating the fields, adding new locations, deleting locations and seemed to be all good :)
Several inline/inliteral methods have been unnested.
The mapview changeEnd event is no longer nested in the mapview decorator but bound to the mapview object as mapview.changeEnd()
This function is bound to the mapview object in the decorator and triggered by the mapview.Map moveend event.
The mapp.layer.changeEnd module has been removed. The changeEnd function has been added to the layer/decorate module. The method will be added to the mapview.Map changeEnd eventlistener for layer with tables or viewport param.
The condition for the layer.tables and viewport param has been removed from the mvt and vector format methods.
The mvt changeEndLoad method has been renamed to wktPropertiesLoad with the documentation linking to the featureFormats-wkt_properties and featureStyle-featureProperties methods to explain the functionality better.
A changeEnd event will be dispatched when a new location is created in the mapview.
The plugins/zoomBtn method documentation has been cleaned up. There was some confusion on params/properties.
The /ui/layers/view module and layersView method has been documented and cleaned up. The zoomToRange and changeEnd methods have been unnested and documented.