Delete simulation polygon

[nickhorspool]

Hi Andreas,

Another piece of feedback from the workshop was that if people made errors with the simulation polygon in step 2, there was no way for them to delete it and they had to restart the simulator app and start again which is pretty annoying for them.

Can you put in a way to delete the polygon, and also some people were able to draw two simulation polygons. this should not be allowed.

Cheers
NIck

[ahocevar]

So we are dealing with a bug and with a feature request here.

Regarding the bug where people are able to draw two simulation polygons, I need exact steps to reproduce it (I was not able to so far).

Regarding the feature request, would you prefer a Delete button in the "Draw or Import" row, or to delete a selected geometry by hitting "Del" when over a feature?

[nickhorspool]

I think a delete button would be consistent with the other polygons so lets
go for that if you are happy with that.

I will try and reproduce the two polygon issue and.

Nick

On Thu, Jul 14, 2011 at 6:10 PM, ahocevar <
[email protected]>wrote:

So we are dealing with a bug and with a feature request here.

Regarding the bug where people are able to draw two simulation polygons, I
need exact steps to reproduce it (I was not able to so far).

Regarding the feature request, would you prefer a Delete button in the
"Draw or Import" row, or to delete a selected geometry by hitting "Del" when
over a feature?

Reply to this email directly or view it on GitHub:
#57 (comment)

[nickhorspool]

As mentioned above, I would be happy with a delete button similar to what is available for the internal polygons.

[bartvde]

Just to clarify, will the delete button delete the simulation area with all its internal polygons, or just the simulation area itself?

[nickhorspool]

Just the simulation area polygon. Not the internal polygons.
On 21 Sep 2011 19:54, "Bart van den Eijnden" <
[email protected]>
wrote:

Just to clarify, will the delete button delete the simulation area with
all its internal polygons, or just the simulation area itself?

Reply to this email directly or view it on GitHub:
#57 (comment)

[bartvde]

Nick, Jeff: how should we handle existing inner polygons? When digitizing the new simulation area they are not re-checked.

[bartvde]

AFAICT the client-side work for this issue has been done. Leaving the ticket open to get an answer to the above question.

[nickhorspool]

You will need to recheck them because they may fall outside or partially
outside the sim polygon which is not allowed.
On 26 Sep 2011 18:05, "Bart van den Eijnden" <
[email protected]>
wrote:

Nick, Jeff: how should we handle existing inner polygons? When digitizing
the new simulation area they are not re-checked.

Reply to this email directly or view it on GitHub:
#57 (comment)

[bartvde]

Thanks for the clarification, I will recheck them in the client.