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

Two-step deletion of features with addresses #6103

Closed
quincylvania opened this issue Mar 25, 2019 · 6 comments
Closed

Two-step deletion of features with addresses #6103

quincylvania opened this issue Mar 25, 2019 · 6 comments
Assignees
Labels
validation An issue with the validation or Q/A code
Milestone

Comments

@quincylvania
Copy link
Collaborator

Mappers should delete defunct points of interest, such as restaurants that go out of business. However, the address tags of these features are likely still valid for geocoding. We should encourage preservation of addresses by having two-step deletion of addressed features:

  1. User clicks delete. All tags are removed except addr:* and the preset is thus downgraded to "Address".
  2. User clicks delete again to fully delete the feature.

The feature would remain selected after step 1, so totally deleting the feature could still be done quickly with just one extra keypress.

Ideally we should find a good place to explain this behavior.

@quincylvania quincylvania added the validation An issue with the validation or Q/A code label Mar 25, 2019
@bhousel
Copy link
Member

bhousel commented Mar 26, 2019

This is a good idea 👍

@quincylvania quincylvania self-assigned this Apr 1, 2019
@quincylvania quincylvania added this to the 2.15.0 milestone Apr 2, 2019
@quincylvania
Copy link
Collaborator Author

I did this in 8779e1a. It works for points, vertices, and buildings. The icon and text make it clear that you're downgrading, but the keyboard shortcut is the same. You can delete immediately after downgrading if you so choose.

downgrade demo

@bhousel
Copy link
Member

bhousel commented Apr 2, 2019

Does this break the “delete a point” step of the walkthrough ?

@quincylvania
Copy link
Collaborator Author

quincylvania commented Apr 2, 2019

Does this break the “delete a point” step of the walkthrough ?

Oh good point. I just tested this and it does break if the user enters address data while editing the cafe, which they can. I'll fix it.

@bhousel
Copy link
Member

bhousel commented Apr 3, 2019

Cool.. It would probably be easiest to silently delete whatever data needs to go away so the user can easily perform the delete step (instead of trying to rewrite the tutorial text - please don’t do this!)

@quincylvania
Copy link
Collaborator Author

@bhousel Yep, that's what I'll do!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
validation An issue with the validation or Q/A code
Projects
None yet
Development

No branches or pull requests

2 participants