Skip to content
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

Merged

Conversation

dbauszus-glx
Copy link
Member

@dbauszus-glx dbauszus-glx commented Oct 23, 2024

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.

@dbauszus-glx dbauszus-glx self-assigned this Oct 23, 2024
@dbauszus-glx dbauszus-glx added the Bug A genuine bug. There must be some form of error exception to work with. label Oct 23, 2024
@dbauszus-glx dbauszus-glx linked an issue Oct 23, 2024 that may be closed by this pull request
@dbauszus-glx dbauszus-glx added Code Issues related to the code structure and performance. Documentation Adding or updating documentation. labels Oct 24, 2024
@dbauszus-glx dbauszus-glx marked this pull request as ready for review October 24, 2024 16:43
@cityremade
Copy link
Member

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.

@dbauszus-glx
Copy link
Member Author

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

Copy link
Contributor

@RobAndrewHurst RobAndrewHurst left a 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!

Copy link
Contributor

@simon-leech simon-leech left a 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:

  1. MVT Layer with a categorized thematic on field_a
  2. Select a location on this layer, and update the value of field_a to a different option in the thematic.
  3. The locations styling is not updated correctly on the mapview, the theming is not updated.
  4. 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

@dbauszus-glx
Copy link
Member Author

@simon-leech I create a separate issue for this. #1624

Copy link

Copy link
Contributor

@AlexanderGeere AlexanderGeere left a 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

Copy link
Contributor

@simon-leech simon-leech left a 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

Copy link
Contributor

@simon-leech simon-leech left a 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 :)

@dbauszus-glx dbauszus-glx merged commit d18842e into GEOLYTIX:main Oct 30, 2024
5 checks passed
@dbauszus-glx dbauszus-glx deleted the mapview-layer-changeend-events branch November 27, 2024 08:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug A genuine bug. There must be some form of error exception to work with. Code Issues related to the code structure and performance. Documentation Adding or updating documentation.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

layer and mapview changeend event methods
5 participants