-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Adjust preset list based on connected or nearby features #4139
Comments
There’s a neverending list of presets we could helpfully suggest, so it’ll be a an area for continuous improvement – and a boon for micromapping. 😄 More connected features:
More nearby features:
|
Good suggestions, @1ec5. Also I want to add, we have traditionally avoided building this kind of OSM tagging logic into iD itself, (as you said, it's a "neverending list of presets"). So the rules for this issue and #4138 should exist in the preset definitions, not in JavaScript code. Something like: {
"icon": "park",
"geometry": [
"point",
"area"
],
"related": {
"contains": [
"leisure/picnic_table",
"amenity/bench",
"amenity/bbq",
"amenity/fountain",
...
],
"nearby": [
"highway/footpath",
...
]
},
"terms": [ ... ],
"tags": {
"leisure": "park"
},
"name": "Park"
} {
"icon": "poi-traffic-signals",
"geometry": [
"vertex"
],
"fields": [
"traffic_signals"
],
"related": {
"connects": [
"highway/trunk",
"highway/primary",
"highway/secondary",
...
]
},
"tags": {
"highway": "traffic_signals"
},
"terms": [ ... ],
"name": "Traffic Signals"
} |
…ered together Display presets related to nearby features as defaults in the preset browser (re: #4139)
Currently the preset list is ordered by most recently used followed by the default common presets. We can improve the ranking of some presets based on connected or nearby features.
Examples:
Connected features
Nearby features (also see everything in #4138)
etc..
cc @1ec5
The text was updated successfully, but these errors were encountered: