----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/121095/#review70236 -----------------------------------------------------------
Perhaps add a framework guy to this request (and other request) as David Faure for example. Without a specific guy I think that nobody will comment them. Regards - Laurent Montel On nov. 11, 2014, 12:12 après-midi, Daniel Vrátil wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/121095/ > ----------------------------------------------------------- > > (Updated nov. 11, 2014, 12:12 après-midi) > > > Review request for KDE Frameworks. > > > Repository: frameworkintegration > > > Description > ------- > > This patch is related to /r/121094, which moves KTextToHTML conversion > utility from KPimUtils to KCoreAddons. Since KCoreAddons can't depend on > KEmoticons needed for smileys conversion, I added the actual KEmoticons code > here, to create a run-time dependency, similar to the KWidgetsAddons-KConfig > dependency for KMessageBox. > > This patch refactors the FrameworkIntegrationPlugin a bit - I split the > KMessageBox-specific code into a separate file, and added a new file with the > KTextToHTMLEmoticonsInterface implementation, as we can't just keep stacking > more and more classes into a single file :-) > > > Diffs > ----- > > CMakeLists.txt 3721bfa > src/integrationplugin/CMakeLists.txt 3395368 > src/integrationplugin/frameworkintegrationplugin.h 6dc6825 > src/integrationplugin/frameworkintegrationplugin.cpp a45ba9d > src/integrationplugin/kmessagebox.h PRE-CREATION > src/integrationplugin/kmessagebox.cpp PRE-CREATION > src/integrationplugin/ktexttohtml.h PRE-CREATION > src/integrationplugin/ktexttohtml.cpp PRE-CREATION > > Diff: https://git.reviewboard.kde.org/r/121095/diff/ > > > Testing > ------- > > Tested with KTextToHTML code from /r/121094 in a QGuiApplication and it seems > to work. > > > Thanks, > > Daniel Vrátil > >
_______________________________________________ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
