> On Sept. 4, 2012, 3:59 p.m., Lasath Fernando wrote: > > Seems like a good patch, I'm impressed at how consistent you've been with > > the rest of the code. (Even with some of my unusual conventions which I > > don't know what state of mind I was in when I came up with). > > > > However, I have one (rather serious) question: what's the point of this?
The point for this patch (and for http://git.reviewboard.kde.org/r/106302/ ) is this cool plugin I'm writing: http://wstaw.org/m/2012/08/30/snapshot92.png http://wstaw.org/m/2012/08/30/snapshot93.png http://wstaw.org/m/2012/08/31/snapshot94.png http://wstaw.org/m/2012/08/31/snapshot95.png If pictures still didn't convince you... I'm using jquery-snippet ( http://www.steamdev.com/snippet/ ), a jQuery syntax highlighting plugin. This requires that a script and a css are included in the code, and then a javascript call is executed to do the real formatting. Yes, the changes that the script do to the html page could be done manually, but I believe it's not worth re-writing it from scratch... Since I believe that we could have more filters later that could require something similar, I made it as generic as possible... I will update both reviews asap. - Daniele Elmo ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/106301/#review18512 ----------------------------------------------------------- On Sept. 1, 2012, 9:18 a.m., Daniele Elmo Domenichelli wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/106301/ > ----------------------------------------------------------- > > (Updated Sept. 1, 2012, 9:18 a.m.) > > > Review request for Telepathy. > > > Description > ------- > > Add a script property to AdiumThemeMessageInfo to be run after appending the > message > Add a string list to Message enable plugins to append scripts > Set the script to the AdiumThemeMessageInfo > Execute the script after appending a message > > > This addresses bug 306020. > http://bugs.kde.org/show_bug.cgi?id=306020 > > > Diffs > ----- > > lib/adium-theme-message-info.h 7b7587ecb7e418258cc6154587dff43f22a86f7b > lib/adium-theme-message-info.cpp e32910f0a165ec66df73e10c5dfcf6fdfc26a9a0 > lib/adium-theme-view.h c3ba08946f081bb054198d4eaeefe19223e1fb3e > lib/adium-theme-view.cpp df2b6d7677176a1385ce1f7eca490766cea7a7b3 > lib/chat-widget.cpp 6c9c7ed7f39f0b9f9ee8fbb49d9fa27d2b6b0857 > lib/logmanager.cpp ac311a20234feb6be961aa4617fe63fa5979f987 > lib/message.h 74321300c9fed3491924de96ffa40eb7e7b9afd6 > lib/message.cpp 74db8e166f710ae37a2e88e057b8bd09d546e634 > > Diff: http://git.reviewboard.kde.org/r/106301/diff/ > > > Testing > ------- > > > Thanks, > > Daniele Elmo Domenichelli > >
_______________________________________________ KDE-Telepathy mailing list [email protected] https://mail.kde.org/mailman/listinfo/kde-telepathy
