Don't show maxspeed if maxspeed:forward &/or maxspeed:backward is set

openstreetmap / iD

🆔 The easy-to-use OpenStreetMap editor in JavaScript.

https://www.openstreetmap.org/edit?editor=id

ISC License

3.34k stars 1.2k forks source link

Don't show maxspeed if maxspeed:forward &/or maxspeed:backward is set #6415

Open JokerGermany opened 5 years ago

JokerGermany commented 5 years ago

To prevent setting accidentally the maxspeed, when maxspeed:forward and/or maxspeed:backward it would be great if maxspeed isn't shown in the div class disclosure-wrap disclosure-wrap-preset_fields when maxspeed:forward and/or maxspeed:backward is set.

1ec5 commented 5 years ago

iD shows a two-row field for cycleway:left and cycleway:right. Perhaps iD could swap in a similar field when maxspeed:left and maxspeed:right are already tagged, but leave the simpler field in place otherwise, since per-direction speed limits are rare by comparison.

BjornRasmussen commented 5 years ago

Why not just improve the field to include forward/backward support just like the bike lanes field at all times?

bhousel commented 5 years ago

Why not just improve the field to include forward/backward support just like the bike lanes field?

We probably will - but it's work someone has to do.