-
-
Notifications
You must be signed in to change notification settings - Fork 358
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
street furniture overlay #5373
street furniture overlay #5373
Conversation
app/src/main/java/de/westnordost/streetcomplete/osm/StreetFurniture.kt
Outdated
Show resolved
Hide resolved
app/src/main/java/de/westnordost/streetcomplete/osm/StreetFurniture.kt
Outdated
Show resolved
Hide resolved
This comment was marked as resolved.
This comment was marked as resolved.
app/src/main/java/de/westnordost/streetcomplete/osm/StreetFurniture.kt
Outdated
Show resolved
Hide resolved
Regarding what should be in the overlay and what not: StreetComplete affords to be more opinionated than iD (presets) when it comes down to which things to support displaying and adding. iD potentially has everything in the presets for which there is a clear documentation in the wiki, no disputes (and someone created a PR for that). For StreetComplete on the other hand, the quest and overlay guidelines continue to apply. For example, storm drains ( Furthermore, certain things should be not included in the overlay that may qualify as street furniture but should rather go into an own, dedicated overlay. One example of these would be A feature which fulfills the StreetComplete guidelines that fits into the overlay but there is no iD preset yet, can still be added to the overlay. It may not be displayed and not be addable for now, but as soon as an iD preset is added, it would be. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A preliminary review:
app/src/main/java/de/westnordost/streetcomplete/quests/shop_type/ShopUtils.kt
Outdated
Show resolved
Hide resolved
app/src/main/java/de/westnordost/streetcomplete/quests/shop_type/ShopUtils.kt
Outdated
Show resolved
Hide resolved
app/src/test/java/de/westnordost/streetcomplete/overlays/street_furniture/FlagpoleMatching.kt
Outdated
Show resolved
Hide resolved
...n/java/de/westnordost/streetcomplete/overlays/street_furniture/StreetFurnitureOverlayForm.kt
Outdated
Show resolved
Hide resolved
.../main/java/de/westnordost/streetcomplete/overlays/street_furniture/StreetFurnitureOverlay.kt
Outdated
Show resolved
Hide resolved
...n/java/de/westnordost/streetcomplete/overlays/street_furniture/StreetFurnitureOverlayForm.kt
Outdated
Show resolved
Hide resolved
...n/java/de/westnordost/streetcomplete/overlays/street_furniture/StreetFurnitureOverlayForm.kt
Outdated
Show resolved
Hide resolved
...n/java/de/westnordost/streetcomplete/overlays/street_furniture/StreetFurnitureOverlayForm.kt
Outdated
Show resolved
Hide resolved
...n/java/de/westnordost/streetcomplete/overlays/street_furniture/StreetFurnitureOverlayForm.kt
Outdated
Show resolved
Hide resolved
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
Also, I have tested it while going home: and this overlay works better than I expected. |
app/src/main/java/de/westnordost/streetcomplete/osm/StreetFurniture.kt
Outdated
Show resolved
Hide resolved
app/src/main/java/de/westnordost/streetcomplete/osm/StreetFurniture.kt
Outdated
Show resolved
Hide resolved
app/src/main/java/de/westnordost/streetcomplete/osm/StreetFurniture.kt
Outdated
Show resolved
Hide resolved
Co-authored-by: Flo Edelmann <git@flo-edelmann.de>
Co-authored-by: Flo Edelmann <git@flo-edelmann.de>
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
Remaining TODOs:
|
when you select already added object marker has bench icon |
I'd like to see |
+1 to taking some decision on split here
It is kind of confusing for people used to I think I would like to have |
"Places" could also be named "Shops & Amenities". But this would be slightly incorrect as it includes more and I think, the association with Also, since "Street Furniture" does not work 100% for the things included plus more importantly, I expect it will be difficult and/or awkward to translate, I am considering just using the word "Things". And while the term "thing" is really fuzzy, everybody would agree that e.g. a "supermarket" or "restaurant" is not a "thing". But a bench is a "thing", isn't it? So there are then Places, Things, (Structures, Roads). |
I'll work on this more today |
So what do you think about the rename to "Things"? Compared with "Street Furniture". Note the list of things that are currently included in the overlay: https://github.com/streetcomplete/StreetComplete/pull/5373/files#diff-5371d38421445449e44a28b1bccef727000adf69d46fb3ed3ae22989b888b227) |
Hm, right, well, this is somewhat of a technical limitation. So, maybe use the plus icon for all? Or maybe have a white dot (inside the green circle) as an icon for the overlay? |
Pedant, does that mean you can't map kiosks as a shop? As an end user, you don't enter them. I can't think of a better simple wording though (apart from usually can be entered).
It seems a bit of a shame we can't just drop a pin on the map to say this is here. I appreciate the footprint is probably more important with these than a building.
It seems a huge shame and rather inefficient that the camp ones (which would trigger additional SC quests) can't be added to the map from within SC.
It seems a bit odd I can do bench->picnic table in the quest, but not in this new overlay, or e.g. phonebox re-purposed to sharing library. Likewise I've found disused post boxes and phoneboxes quite often. The thing is still there, but it doesn't serve its usual purpose.
It seems a reasonable compromise to me, again I can't think of a different all-encompassing name and it fits well with places.
I'm pretty certain I've seen dolphins, but didn't know they were called that. |
app/src/main/java/de/westnordost/streetcomplete/osm/StreetFurniture.kt
Outdated
Show resolved
Hide resolved
Yeah, you cannot enter a kiosk, but just like (street) food stalls that only have outdoor seating (if at all) I would still rather allocate them to places than things. Regarding your remarks: It is unfortunate, but do you appreciate the reasoning behind that? |
fixes #4912