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

Building part partial fill narrower #7257

Closed
IgorEliezer opened this issue Jan 17, 2020 · 4 comments
Closed

Building part partial fill narrower #7257

IgorEliezer opened this issue Jan 17, 2020 · 4 comments
Labels
usability An issue with ease-of-use or design

Comments

@IgorEliezer
Copy link

Please, could you make the building part fill a bit narrower? Perhaps, slightly narrower than the half width of building outline fill (in red).

image
https://www.openstreetmap.org/changeset/79578964

Today I had to fix a changetset of a new mapper who wanted to reach the main building outline to add extra info and mistakenly deleted the building parts thinking they were separated buildings.

Regards,

Igor

@quincylvania
Copy link
Collaborator

@IgorEliezer Hmm the issue is that we actually have three levels of nested areas: indoor rooms are narrower than building parts. We need to leave room these.

We're tracking #6204 to render building parts differently plus #2225 and #5520 to make selecting overlapping areas easier. Those should address the underlying concern, no?

@quincylvania quincylvania added considering Not Actionable - still considering if this is something we want usability An issue with ease-of-use or design labels Jan 17, 2020
@quincylvania
Copy link
Collaborator

Here are the three levels together.

Screen Shot 2020-01-17 at 3 07 13 PM

@IgorEliezer
Copy link
Author

IgorEliezer commented Jan 17, 2020

Hmm the issue is that we actually have three levels of nested areas: indoor rooms are narrower than building parts.

Oh I didn't know there were 3 levels of building fill. As you said here, using a pattern can be a good solution.

If you judge that this issue overlaps the other issues, feel free to close it. 👍

@quincylvania
Copy link
Collaborator

@IgorEliezer Great, thanks for understanding! I agree we should make area selection better in iD.

@quincylvania quincylvania removed the considering Not Actionable - still considering if this is something we want label Jan 17, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
usability An issue with ease-of-use or design
Projects
None yet
Development

No branches or pull requests

2 participants