every_door icon indicating copy to clipboard operation
every_door copied to clipboard

[Request] Add support for Waterway Dam

Open Coehill opened this issue 1 year ago • 4 comments

waterway=dam would be a useful addition for those traveling in canoes, kayaks, boats, etc.

Coehill avatar Jul 03 '24 15:07 Coehill

Can you specify what seems to be the problem? I seem to have the option to add waterway=dam in EveryDoor 5.1 (Android)

mnalis avatar Jul 07 '24 02:07 mnalis

Can you specify what seems to be the problem? I seem to have the option to add waterway=dam in EveryDoor 5.1 (Android)

Sure! When trying to add waterway=dam, there is a yellow warning "this object type is not supported". Additionally, Dam points are not displayed or editable on the map (see comparison of Every Door vs. OsmAnd~ below). Hopefully this helps.

78d9264b-1862-4ad2-9471-0c68767a08c3

70aa1df7-a912-47d3-b5c3-ff93ccd0582e

0319904a-54bd-4f41-a166-69c8981b64b9

Coehill avatar Jul 10 '24 12:07 Coehill

Relevant code: https://github.com/Zverik/every_door/blob/ed9bf4c47548920ac668be57484dc91ece6612a3/lib/helpers/poi_warnings.dart#L63

https://github.com/Zverik/every_door/blob/ed9bf4c47548920ac668be57484dc91ece6612a3/lib/helpers/good_tags.dart#L267

I think it just needs dam to be added in an if statement in good_tags.

Coehill avatar Jul 22 '24 20:07 Coehill

But waterway=dam node should snap on waterway ways, though, right? I'm not sure if just adding to good_tags is going to do that...

mnalis avatar Jul 22 '24 20:07 mnalis

This is actually sensible. 5.2 will support waterway features, and will snap most of them to nearby waterways.

Zverik avatar Sep 21 '24 17:09 Zverik