Kovoschiz's Comments
| Changeset | When | Comment |
|---|---|---|
| 180715131 | 2. The `=cliff` split were correct in separating. `=embankment` shouldn't enclose the whole section. It's for the slanted slopes on the western and eastern sides only. North and south ends should be `=retaining_wall` for vertically. |
|
| 180715131 | `level=` shouldn't be added to `building=` |
|
| 180700113 | Don't use brackets as descriptor labels on `name=` , which is for proper names only. Use `alt_name=` + `ref=` |
|
| 180663681 | Please don't delete it, as it contains an address. Change to ` was:amenity= ` ` disused:amenity= ` / ` abandoned:amenity= ` |
|
| 180673935 | You are still mixing `=informal` and `=unmarked` |
|
| 180660380 | 3. There's no need to blindly align exactly to some imagery at some version. There will be offsets. Eg it matches old Esri Clarity. |
|
| 180660380 | 0. Please use descriptive upload comment
|
|
| 180617013 | Slip roads `=bridge` should not be drawn parallel. It should follow joints, with perpendicular ends. Except perhaps eastbound expansion, but it's stitched together already, and jointless above.
|
|
| 180617013 | You made conflicting layers. Slip roads passing below `layer=2` IEC is `layer=1` |
|
| 180578807 | If you want to do phased openings, you need to use `construction:junction=roundabout` |
|
| 180578509 | `name=` is for what it would be after completion. Not project or contract title, which are descriptive labels. |
|
| 180578509 | Please don't artificially compose names. `name=` is for proper names only. They are already added separately inside. |
|
| 180549411 | Can you stop promoting small, sparse, lightly populated, insignificant villages with no own facilities? That's one reason why `=suburb` is used to distinguish between distinct villages, and large areas. |
|
| 180542128 | If you want, you might try `addr:housename=` , but that will be opposed for not being a postal address. While `is_in:housename=` is often misunderstood as deprecated, here it's indeed unneeded as there's a spatial relationship. You shouldn't add such attributes to `=fitness_station` , `=pitch` inside either. |
|
| 180542128 | Don't add the same `name=` to `=playground` inside a `=park` (where there can be multiple of them, as it's used similar to `=pitch` for patches of playgrounds) |
|
| 180533601 | Might be an SC bug. But I can't imagine why it does this. Need to test the app on Android. |
|
| 180456600 | Be bold in what you create. Be cautious in what you change. https://en.wikipedia.org/wiki/G._K._Chesterton#Chesterton's_fence
|
|
| 180456600 | You did cause damage by removing the `building=` there. I'm pointing this out. If you don't want to read them, ignore it. But don't complaint if others change your work. |
|
| 180456600 | 0. Please use descriptive upload comments, or split your changeset
|
|
| 180471684 | And you still made it inconsistent, as there are `*:suburb=` |