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

Time series plotter: Allow adding groundwater nodes and lines easily #928

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

leendertvanwolfswinkel
Copy link
Collaborator

In groundwater models, the surface water nodes/lines are at the same location as the groundwater nodes/lines. In the current implementation, only the first feature (i.e. the surface water feature) will be added to the plot. I changed this so that all features within a certain precision from the mouse are added to the plot.

However, if you zoom out very far and click on the map, it will freeze the Modeller Interface for quite a while and add loads of lines to your plot. I propose the following solution:

  • If the number of items that are to added to the plot exceeds 10, do not add any and show the following message as a messagebar: "Please zoom in and try again. To add more than 10 items to the plot at once, use the 'Add all selected' option."

@leendertvanwolfswinkel leendertvanwolfswinkel changed the base branch from master to refactor October 11, 2023 15:52
@arjanverkerk
Copy link
Contributor

Without trying or testing anything, in the water balance we get a popup in which you can select from multiple geometries in multiple layers if there are more candidates within the search range. Could we use that behaviour here as well, or is it desirable to be able to select multiple features at once.
select

Base automatically changed from refactor to master July 18, 2024 09: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