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

Added a control to allow clearing selections #273

Merged
merged 3 commits into from
Nov 13, 2017

Conversation

theduckylittle
Copy link
Member

After all queries have been removed from the selection tab,
the selection polygon would continue to be displayed.

This new functionality fixes that issue by adding a button
allowing the user to clear the polygon. This was done as a bit
of a compromise. Ostensibly, a complex selection area may need to be
re-used even if the user wants to clear teh current results. Automatically
removing the selection area would've made that impossible.

Steps to use:

  1. run a query with a drawn selection feature.
  2. click "delete" / trash-can icon on the results.
  3. See that there is now a polygon on display.
  4. click the 'clear' button in the service manager tab.

refs: #257

After all queries have been removed from the selection tab,
the selection polygon would continue to be displayed.

This new functionality fixes that issue by adding a button
allowing the user to clear the polygon.  This was done as a bit
of a compromise. Ostensibly, a complex selection area may need to be
re-used even if the user wants to clear teh current results.  Automatically
removing the selection area would've made that impossible.

Steps to use:
1. run a query with a drawn selection feature.
2. click "delete" / trash-can icon on the results.
3. See that there is now a polygon on display.
4. click the 'clear' button in the service manager tab.
@elil
Copy link
Member

elil commented Nov 10, 2017

This seems like fine grained control! Probably appreciated by those who know what it is and use it, probably ignored by many. Out of curiosity, what happens when you use this order for the steps?

  1. run a query with a drawn selection feature.
  2. click the 'clear' button in the service manager tab.
  3. See that there is now a polygon on display.
  4. click "delete" / trash-can icon on the results.

@theduckylittle
Copy link
Member Author

The clear button is disabled when there is no polygon available to clear. If you do hit the clear button while drawing, it will just remove the currently in progress selection.

@elil
Copy link
Member

elil commented Nov 10, 2017

👍

@klassenjs klassenjs merged commit 12c3658 into geomoose:master Nov 13, 2017
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.

3 participants