> On 2010-11-07 21:36:12, Andrea Diamantini wrote: > > If your testing against this patch work, I'm ok on merging. > > Panagiotis Papadopoulos wrote: > yeah it does work here.
Ok. Can you please merge this patch and sign this as "closed --> submitted"? Thanks :) - Andrea ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/100123/#review326 ----------------------------------------------------------- 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
