<html>
<body>
<div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
<table bgcolor="#f9f3c9" width="100%" cellpadding="8" style="border: 1px #c9c399 solid;">
<tr>
<td>
This is an automatically generated e-mail. To reply, visit:
<a href="http://git.reviewboard.kde.org/r/100215/">http://git.reviewboard.kde.org/r/100215/</a>
</td>
</tr>
</table>
<br />
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<p style="margin-top: 0;">On January 5th, 2011, 10:32 a.m., <b>Jon Ander Peñalba</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">I would like to have the option of opening all the links in the selected text in new tabs, but I don't like how this patch implements it.
This patch only takes into account the text (if it starts with http://, https:// or www. it's a link), it doesn't look for actual links, and if they are both present this is not at all intuitive.
I would like to see this working only with actual links or not implemented at all.</pre>
</blockquote>
</blockquote>
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">I would tend to agree with this: I believe actual links are more important than text that seems to be a valid url, and links will most likely not have http:// in their text. The problem is that there is no nice API in QtWebKit to to that at the moment, and using the selection and ranges in Javascript requires Javascript to be enabled, which we can't guarantee.
Andrea has a point in his comment from December 17th: do we really need this ? As a side note: since a Javascript approach would work, this could be done quite simply via a bookmarklet.</pre>
<br />
<p>- Pierre</p>
<br />
<p>On December 17th, 2010, 5:12 p.m., Panagiotis Papadopoulos wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://git.reviewboard.kde.orgrb/images/review_request_box_top_bg.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
<tr>
<td>
<div>Review request for rekonq.</div>
<div>By Panagiotis Papadopoulos.</div>
<p style="color: grey;"><i>Updated 2010-12-17 17:12:32</i></p>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Description </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">When you select an area of text on the page that contains more than one link,
it would be very useful if you were given a right-click option to open those
links in new tabs. This is a very useful feature in firefox. An option to
open them in a single new window would also be nice, although not important
since it is easy enough to detach the tab first.
Taken from the Wish on bko
Patch by Andrzej G?owi?ski </pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Testing </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">works here</pre>
</td>
</tr>
</table>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Bugs: </b>
<a href="/show_bug.cgi?id=248863">248863</a>
</div>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>src/webview.h <span style="color: grey">(9518b74)</span></li>
<li>src/webview.cpp <span style="color: grey">(aeaf92b)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/100215/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>