[rekonq] Re: Review Request: opening plain text emails in email application

[Andrea Diamantini]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/100123/#review242
-----------------------------------------------------------


Things to check:
1) does KDE have some classes to check if a text is a valid email address (instead of parsing it everytime with a regexp)?
2) does not the created url need a scheme? (eg: mailto:pano at rekonq.org)?

- Andrea


On 2010-10-31 18:53:33, Panagiotis Papadopoulos wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/100123/
> -----------------------------------------------------------
> 
> (Updated 2010-10-31 18:53:33)
> 
> 
> Review request for rekonq.
> 
> 
> Summary
> -------
> 
> This adds support for opening a plain text email address in an appropriate email application.
> 
> I need someone that can help me with the TODO part… Currently when you execute the action to send an email, rekonq will open up a new tab first (due to the usage of the openLinkInNewTab slot). IMHO this is not necessary…
> 
> (note: I am no developer, my programming skills are *very* low :-D This is mostly a “copy, paste and modify” job :-))
> 
> 
> Diffs
> -----
> 
>   src/webview.cpp 8be744a 
> 
> Diff: http://git.reviewboard.kde.org/r/100123/diff
> 
> 
> Testing
> -------
> 
> worked here a week ago, but due to some weird problem on my installation (kmailservice failing to be started) it does not work today… weird 
> 
> 
> Thanks,
> 
> Panagiotis
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.kde.org/pipermail/rekonq/attachments/20101031/c0db1456/attachment.htm