-
Notifications
You must be signed in to change notification settings - Fork 24
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
Attributes pop-up improvements #1019
Comments
It looks like you want to accumulate and aggregate results from potentially totally different features sets. Keep in mind that this tools already aggregate overlapping features values(see bellow), and handle most WMS queryFeatures returned values. Two options for this :
Both cases require a medium refactor of this tool, for covering all possible cases, and handle merging results. The second option seems simpler, as it could be handled upstream – before building the widget. Another idea :
|
If I understand @thomaspiller's proposal correctly, the idea is not to aggregate or accumulate the results of different features, but rather the following:
Objectively, I am fine with both behaviors. Otherwise, I approve of requests number 1 (very important) and 3:
|
In my opinion, number 2 is the most important feature.
If this is too complex to have systematically, is there a way to have this behavior at least when the clicked layers do not change between two consecutive clicks? [e.g. t: 0, clicked layers A; t: 1, clicked layers: A]. [e.g. t: 0, clicked layers A, B; t: 1, clicked layers: A, B] Number 3 is also cool to have. |
It is planned to refactor the pop-up taking into account the specifications listed below. The attributes of the clicked features will be displayed in a table format allowing a better visualization of the data. Filters by value will be preserved.
Here is a prototype of what is planned to be developed: https://fxi.io/demo_table_attribute/
Initial post
The user experience of the pop-up could be improved by implementing the following changes (based on user feedback, in priority order):
The text was updated successfully, but these errors were encountered: