> On 2010-10-31 22:36:03, Andrea Diamantini wrote: > > 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])?
to 1) Searching for “email” on the KDE API website, the only usable thing I found was: http://api.kde.org/4.x-api/kdepimlibs-apidocs/kpimutils/html/group__emailvalidation.html#gadf2a1bc99ab8a016c1c782ac574a58de I do not think that adding a dependency to kdepimlibs just for such a tiny feature is a good idea though^^ to 2) due to rohans previous work on the protocol handling, it seems to not be needed… - Panagiotis ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/100123/#review242 ----------------------------------------------------------- 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
