After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 784072 - Show mapping notes in edit dialog
Show mapping notes in edit dialog
Status: RESOLVED FIXED
Product: gnome-maps
Classification: Applications
Component: general
git master
Other Linux
: Normal enhancement
: ---
Assigned To: gnome-maps-maint
gnome-maps-maint
Depends on:
Blocks:
 
 
Reported: 2017-06-22 00:21 UTC by Robert Ancell
Modified: 2017-07-24 10:47 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
osmEditDialog: Add edit/add of notes (2.53 KB, patch)
2017-06-22 00:21 UTC, Robert Ancell
none Details | Review
Screenshot showing notes in edit dialog (95.09 KB, image/png)
2017-06-22 00:24 UTC, Robert Ancell
  Details
osmEditDialog: Add edit/add of notes (2.71 KB, patch)
2017-07-09 01:49 UTC, Robert Ancell
committed Details | Review

Description Robert Ancell 2017-06-22 00:21:05 UTC
OpenStreetMap has a 'note' tag [1] which is used "to inform other mappers about non-obvious information about an element, the author's intent when creating it, or hints for further improvement".

Attached patch makes the notes show when editing points of interest.

[1] https://wiki.openstreetmap.org/wiki/Key:note
Comment 1 Robert Ancell 2017-06-22 00:21:47 UTC
Created attachment 354209 [details] [review]
osmEditDialog: Add edit/add of notes
Comment 2 Robert Ancell 2017-06-22 00:24:44 UTC
Created attachment 354210 [details]
Screenshot showing notes in edit dialog
Comment 3 Robert Ancell 2017-06-22 00:26:01 UTC
The notes should possibly be shown in a GtkTextView instead of GtkEntry, but I figured we should at least show them first and enhance it later if necessary.
Comment 4 Marcus Lundblad 2017-06-22 20:35:27 UTC
Yeah, a text view would probably be even better, but I guess it would need some thought. I guess just adding it to the present edit view wouldn't fit too well. Maybe an option could be to have some way to expand a text entry to reveal a full text view. But for now, I think we could go for the simple text entry. I wonder though if this field should have a hint explaining that this note is meant for internal edit purposes (i.e. not typically shown on maps, and not a general presentation of the object)?
Comment 5 Robert Ancell 2017-07-09 01:49:24 UTC
Created attachment 355198 [details] [review]
osmEditDialog: Add edit/add of notes

Add hint about usage of note field.
Comment 6 Marcus Lundblad 2017-07-24 09:31:21 UTC
Review of attachment 355198 [details] [review]:

LGTM