Kovoschiz's Comments
Changeset | When | Comment |
---|---|---|
156035746 | 11 months ago | 1. Please don't blindly merge different sections. They are split for a reason,
|
152586882 | 12 months ago | Reminder: `name=` and `place=` are the existing common name and concepts in public use, not what you think should be correct or proper. Put historic names and locations in OpenHistoricalMap, not OSM. Discuss with the community, where the Discord server (https://discord.com/invite/CyG9zte) has been made available in OSM Community Index for a long time already, and there is a wiki project (osm.wiki/Hong_Kong). |
155787407 | 12 months ago | 4. Please don't delete `landuse=` when you draw `building=` . They are different aspects. |
155768420 | 12 months ago | 1.
|
155768420 | 12 months ago | 1. Please don't add such descriptor label to `name=` , which is for proper names only. It's not necessary to add `description=` , or anything here either.
|
155787407 | 12 months ago | 3. As mentioned before, please don't delete something that you haven't even drawn back |
155787407 | 12 months ago | 1. Please don't use `building=warehouse` for the entire land. The `landuse=industrial` should be kept.
|
155786452 | 12 months ago | Please don't directly delete something that is being reconstructed. You should change it to `highway=construction` . |
155767884 | 12 months ago | Please don't use such descriptor labels in `name=` , which is for proper names only. `description=` has already been added. |
152523735 | 12 months ago | Please only use `building=office` for buildings. `office=` should be used for actually offices inside. |
155565161 | 12 months ago | Please don't delete diagonal `=footway` on a scramble crosswalk. You are removing info. |
150326255 | 12 months ago | 2. You need to enclose `building:part=` inside the `building=` . If they are outside for any reason, you need to create a `type=building` with them as `part` and `outline` |
155542787 | 12 months ago | Please don't merge sections of `=footway` around `=steps` . They are either top & bottom, or the intermediate resting platforms. |
155517592 | 12 months ago | Please don't blindly merge different sections. You removed a `covered=yes` here. |
155514085 | 12 months ago | As mentioned before, please don't change `=traffic_signals` to `=crossing` . You need to adjust them to create a separate `=crossing` point. |
155490974 | 12 months ago | Please don't move a `=give_way` upstream of a `=crossing` . This is basically never the case, and is misleading whether it is for the crosswalk. |
155489775 | 12 months ago | Please don't delete `crossing=no` . They indicate where there isn't a crosswalk. |
155424527 | 12 months ago | Please don't delete large polygons without fully replacing it. For large edits, you need to consider how to work methodically. Eg cut it up. |
155326896 | 12 months ago | As mentioned before, please don't simply disconnect `=footway` or `=steps` from the eg `highway=tertiary` roadway. This breaks routing. Aside from drawing `=footway` across `=crossing` , you still need to draw an artificial `=footway` when there's no `=service` road for the routing to coincidentally connect back. In the case of `=steps` , it also more accurately shows how it is connected, similar to how `bridge=` isn't directly connected with intersections. |
155304301 | 12 months ago | `short_name=` should be used to further remove the generic form of the proper name, eg `name:en=Mun Tung Estate` can be `short_name:en=Mun Tung` |