----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/121095/#review70233 -----------------------------------------------------------
src/integrationplugin/frameworkintegrationplugin.cpp <https://git.reviewboard.kde.org/r/121095/#comment49103> Perhaps we can remove this #if QT_VERSION no ?:) We use QT> 5.0 :) - Laurent Montel On nov. 10, 2014, 3:59 après-midi, Dan Vrátil wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/121095/ > ----------------------------------------------------------- > > (Updated nov. 10, 2014, 3:59 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, > > Dan Vrátil > >
_______________________________________________ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
