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

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

Open
JokerGermany opened this issue May 23, 2019 · 3 comments
Open
Labels
considering Not Actionable - still considering if this is something we want field An issue with a field in the user interface

Comments

@JokerGermany
Copy link

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.

@bhousel bhousel added considering Not Actionable - still considering if this is something we want field An issue with a field in the user interface labels May 23, 2019
@1ec5
Copy link
Collaborator

1ec5 commented May 24, 2019

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
Copy link
Contributor

BjornRasmussen commented May 24, 2019

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

@bhousel
Copy link
Member

bhousel commented May 24, 2019

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
considering Not Actionable - still considering if this is something we want field An issue with a field in the user interface
Projects
None yet
Development

No branches or pull requests

4 participants