Modify

Opened 9 years ago

Closed 6 years ago

Last modified 6 years ago

#11217 closed enhancement (fixed)

support urls in notes

Reported by: skyper Owned by: team
Priority: normal Milestone: 17.11
Component: Core notes Version:
Keywords: Cc: Klumbumbus

Description (last modified by skyper)

Not sure if it is within the scope of the plugin. If not it would be an enhancement of "Core notes".

I often find URLs within a note but cannot click nor copy it to open in a web browser.

Attachments (5)

11217.patch (1.6 KB ) - added by simon04 9 years ago.
2015-04-15-213157_1198x1899_scrot.png (1.1 KB ) - added by simon04 9 years ago.
First attempt using GettingStarted.LinkGeneral: does not look as expected …
josm_notes.png (9.8 KB ) - added by Don-vip 6 years ago.
note_escape.gif (1.7 MB ) - added by Klumbumbus 6 years ago.
notes_arrow.gif (555.9 KB ) - added by Klumbumbus 6 years ago.

Change History (33)

comment:1 by Don-vip, 9 years ago

Component: Plugin tag2linkCore notes
Keywords: notes removed
Owner: changed from Don-vip to ToeBee

comment:2 by skyper, 9 years ago

Description: modified (diff)

by simon04, 9 years ago

Attachment: 11217.patch added

comment:3 by simon04, 9 years ago

Summary: support urls in notes[Patch] support urls in notes

An idea would be to copy the note URL to the clipboard when right clicking the tooltip, see attachment:11217.patch

comment:4 by bastiK, 9 years ago

Can we use GettingStarted.LinkGeneral?

by simon04, 9 years ago

First attempt using GettingStarted.LinkGeneral: does not look as expected …

comment:5 by Klumbumbus, 9 years ago

Cc: Klumbumbus added

comment:6 by ToeBee, 9 years ago

Technically speaking, the patch and this ticket are solving two different problems. The ticket is asking for URLs in the note text to be clickable. The patch makes it easier to open a given note on osm.org. If note text could be made clickable, the "note xxx" link at the top of the note could be turned into a URL to open on osm.org. The problem I ran into is that the note text is rendered as a tooltip which isn't really intended to be clicked on.

comment:7 by stoecker, 9 years ago

Milestone: 15.05

comment:8 by Don-vip, 9 years ago

Milestone: 15.0515.06

comment:9 by Don-vip, 9 years ago

Milestone: 15.0615.07

skip milestone 15.06

comment:10 by simon04, 9 years ago

In 8503/josm:

see #11217 - Notes: copy note URL to clipboard when right clicking tooltip

comment:11 by simon04, 9 years ago

Summary: [Patch] support urls in notessupport urls in notes

comment:6 still applies: links within notes can still not be opened/copied.

in reply to:  10 comment:12 by Klumbumbus, 9 years ago

Replying to simon04:

In 8503/josm:

see #11217 - Notes: copy note URL to clipboard when right clicking tooltip

This copies the api url. Wouldn't it be better if it copies the url, which opens the note at osm.org?

So,

http://www.openstreetmap.org/note/377327

instead of

https://api.openstreetmap.org/api/0.6/notes/377327

comment:13 by Don-vip, 9 years ago

Milestone: 15.0715.08

Milestone renamed

comment:14 by Don-vip, 9 years ago

Milestone: 15.0815.09

comment:15 by simon04, 9 years ago

Milestone: 15.09
Owner: changed from ToeBee to team

I don't plan implementing this any time soon …

comment:16 by Don-vip, 6 years ago

Milestone: 17.11

comment:17 by Don-vip, 6 years ago

Resolution: fixed
Status: newclosed

In 13111/josm:

fix #11217, fix #12623 - major rework of notes tooltips:

  • display clickable links
  • allow to copy text from notes, including comments

by Don-vip, 6 years ago

Attachment: josm_notes.png added

comment:18 by Don-vip, 6 years ago

Here's the result:


comment:19 by Klumbumbus, 6 years ago

This means copying the notes api url (via right click on the popup) is no longer available, right?
(Just for updating Help/Dialog/Notes.)

by Klumbumbus, 6 years ago

Attachment: note_escape.gif added

comment:20 by Klumbumbus, 6 years ago

The notes Popup is now somehow on top of the JOSM window and can be moved out of the mapview, which looks a bit weird.


Last edited 6 years ago by Klumbumbus (previous) (diff)

comment:21 by Don-vip, 6 years ago

I'm not sure I can display it behind the toolbars and panels with the new system. I could however hide the popup if displayed outside of JOSM...

comment:22 by Don-vip, 6 years ago

In 13117/josm:

see #11217, fix #15541 - hide note tooltip when selected note is no longer visible + drop old url copy code (does not work anymore with new system)

comment:23 by Klumbumbus, 6 years ago

Resolution: fixed
Status: closedreopened

There is one more regression. It is not possible anmore to cycle through the notes list with the arrow keys. I work a lot on handling and closing notes and this feature is pretty helpful as then you can quickly see the full texts of all the notes in the list.

Last edited 6 years ago by Klumbumbus (previous) (diff)

by Klumbumbus, 6 years ago

Attachment: notes_arrow.gif added

comment:24 by Klumbumbus, 6 years ago

And one more regression is that shortcuts don't work anymore after clicking on a note. So my work flow is often:

  1. double click a note
  2. press shortcut for Help/Action/DownloadOsmInView

The shortcut doesn't work, you need to click somewhere else first.

comment:25 by Don-vip, 6 years ago

Resolution: fixed
Status: reopenedclosed

In 13118/josm:

fix #11217 - do not steel focus when the selected note changes

in reply to:  25 comment:26 by Klumbumbus, 6 years ago

Replying to Don-vip:

In 13118/josm:

fix #11217 - do not steel focus when the selected note changes

Mouse scroll to zoom the mapview doesn't work if the mousepointer is on the note tooltip. Can this be fixed easily too?

comment:27 by Don-vip, 6 years ago

In 13126/josm:

see #11217 - forward note tooltips mouse wheel scroll events to map frame

comment:28 by Don-vip, 6 years ago

In 13128/josm:

see #15550, see #11217 - better detection of URLs in notes

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain team.
as The resolution will be set.
The resolution will be deleted. Next status will be 'reopened'.

Add Comment


E-mail address and name can be saved in the Preferences .
 
Note: See TracTickets for help on using tickets.