----------------------------------------------------------- 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:[email protected])? - 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 > >
_______________________________________________ rekonq mailing list [email protected] https://mail.kde.org/mailman/listinfo/rekonq
