Review Request: Improve link highlighting in AdiumThemeView

Alexander Richardson alex.richardson at gmx.de
Wed Jul 20 22:15:41 CEST 2011


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

Review request for Telepathy.


Summary
-------

Also highlight links that don't start with "<protocol>://", but also links that start with "www.".
Also update regular expression to use not whitespace instead of listing all whitespace characters explicitly


Diffs
-----

  lib/adium-theme-view.cpp 9de0d65 

Diff: http://git.reviewboard.kde.org/r/102024/diff


Testing
-------

Previously "www.kde.org" was not highlighted and clickable, now it is.
Any other links starting with a protocol still work as before.


Thanks,

Alexander

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.kde.org/pipermail/kde-telepathy/attachments/20110720/5da68415/attachment.htm 


More information about the KDE-Telepathy mailing list