Close context menu in tutorial when label needs to be deleted

[jonfroehlich]

[jonfroehlich]

I was also slightly confused by the instruction because when I first received it, there were two X's (one on the pop-up dialog and the other on the label itself):

[misaugstad]

I always label curb ramps at the bottom-middle... but tutorial doesn't seem to like this

As of #3210 this is now fixed. For some reason the ideal location for most of the labels was set above the thing being labeled. I slightly adjusted them to be more in line with what we'd expect, so this now works.

I was also slightly confused by the instruction because when I first received it, there were two X's (one on the pop-up dialog and the other on the label itself):

I think that we should probably automatically close the context menu if the label is in the wrong place and we want the user to remove it. This deals with there being two X's, plus we just don't need to show this extra stuff on the screen when we want them to delete it anyway. Going to change the title for the issue to go after this.

[misaugstad]

Maybe we even prevent the context menu from opening at all in this case?

[jonfroehlich]

This seems right to me:

I think that we should probably automatically close the context menu if the label is in the wrong place and we want the user to remove it. This deals with there being two X's, plus we just don't need to show this extra stuff on the screen when we want them to delete it anyway. Going to change the title for the issue to go after this.