Roundup Tracker - Issues

Issue 669789

classification
Multi-line links in message get broken
Type: Severity: normal
Components: Web interface Versions:
process
Status: closed wont fix
:
: richard : jkew, richard
Priority: normal :

Created on 2003-01-17 15:37 by jkew, last changed 2003-04-27 03:37 by richard.

Messages
msg596 Author: [hidden] (jkew) Date: 2003-01-17 15:37
0.5.4 detects and makes into links URLs of the 
form http://stuff/stuff/...

Unfortunately, long links often get broken across 
multiple lines by the wrap="hard" on the Change 
Note textarea. The detection code then makes the 
first line of the URL into a link, leaving the second 
and subsequent lines alone.

For example, I cannot link to SourceForge tracker 
items from issue text as the SourceForge URLs are 
too long to fit on a single line.

I'm not sure this is solvable in the general case, but 
we might be able to do slightly better for common 
cases by using the following rules for link 
detection:

* a space always identifies the end of a link
* end-of-message always identifies the end of a 
link
* a blank line (a line containing only a newline) 
always identifies the end of a link
* a newline does *not* identify the end of a link

although this may rely too heavily on my 
assumption that browsers, when honouring the 
wrap="hard" attribute, leave trailing spaces at end-
of-line when wrapping at word boundaries but do 
not add trailing spaces when breaking long words 
across lines.

The other solution -- and tying back into 
the "Opera doesn't handle wrap=hard" bug -- 
would be for Roundup, not the browser, to perform 
text wrapping. This would mean that Roundup gets 
the links unadulterated by extraneous linebreaks.
msg597 Author: [hidden] (richard) Date: 2003-01-24 06:31
Logged In: YES 
user_id=6405

I'd really rather that Roundup not try to second-guess the browser :( 
 
msg598 Author: [hidden] (richard) Date: 2003-04-27 03:37
Logged In: YES 
user_id=6405

The result of long(ish) on and off discussion is that this will not be 
"fixed" as it's not something that may be reasonably solved by 
Roundup code. 
 
History
Date User Action Args
2003-01-17 15:37:42jkewcreate