Opened 4 years ago
Closed 2 years ago
#3665 closed enhancement (fixed)
Showing waypoints with name and note/comment
| Reported by: | mmehl | Owned by: | team |
|---|---|---|---|
| Priority: | normal | Component: | Core |
| Version: | Keywords: | waypoints | |
| Cc: | michael.mehl@… |
Description
While mapping I sometimes create a waypoint and add a note/comment to this way point.
It would be nice to see the name and the note/comment when loading the waypoints into JOSM.
The patch below, adds the note/comment, if a comment is available.
Index: src/org/openstreetmap/josm/gui/layer/markerlayer/Marker.java
===================================================================
--- src/org/openstreetmap/josm/gui/layer/markerlayer/Marker.java (revision 2252)
+++ src/org/openstreetmap/josm/gui/layer/markerlayer/Marker.java (working copy)
@@ -117,6 +117,10 @@
String name_desc = "";
if (wpt.attr.containsKey("name")) {
name_desc = wpt.getString("name");
+ if (wpt.attr.containsKey("desc")) {
+ name_desc+=", ";
+ name_desc+=wpt.getString("desc");
+ }
} else if (wpt.attr.containsKey("desc")) {
name_desc = wpt.getString("desc");
}
Attachments (1)
Change History (11)
Changed 4 years ago by mmehl
comment:1 Changed 4 years ago by mmehl
- Cc michael.mehl@… added
comment:2 Changed 4 years ago by mmehl
- Summary changed from Showing waypoints with name and note/comment to [Patch] Showing waypoints with name and note/comment
comment:3 Changed 4 years ago by mmehl
- Summary changed from [Patch] Showing waypoints with name and note/comment to [PATCH] Showing waypoints with name and note/comment
comment:4 Changed 4 years ago by stoecker
I'm not really sure that this is the right approach. What if some devices add automated comments and users get a cluttered display due to much useless stuff.
comment:5 Changed 4 years ago by Gubaer
- Resolution set to fixed
- Status changed from new to closed
comment:6 Changed 4 years ago by andrewpmk
My device (Garmin Legend HCx) appears to add the date/time as comment. Could you please add a toggle in preferences so that this isn't too distracting?
comment:7 Changed 4 years ago by anonymous
- Resolution fixed deleted
- Status changed from closed to reopened
The usability of this feature is not yet optimal (see remarks of stoecker and andrewpmk).
I don't think preferences are a good solution, because typical preferences are used as a compromise, when no better solution is found.
Maybe something like showing more details when clicking (or mouse over) on the waypoint ...
comment:8 Changed 4 years ago by stoecker
comment:9 Changed 4 years ago by stoecker
- Summary changed from [PATCH] Showing waypoints with name and note/comment to Showing waypoints with name and note/comment
comment:10 Changed 2 years ago by mjulius
- Resolution set to fixed
- Status changed from reopened to closed
I believe this has been fixed in r3396.



Patch