Re: Review Request 121095: FrameworkIntegration: Add KTextToHTML emoticons support to FrameworkIntegrationPlugin

> On Nov. 11, 2014, 2:39 p.m., Albert Astals Cid wrote: > > I sincerely think this is a *VERY BAD* idea. I don't want my app behaving > > differently depending if a third party misterious compononent that is not > > documented anywhere is installed or not. If you have a dependency, well put >

Re: Review Request 121090: Move kio-mtp to kio-extras

> On Nov. 12, 2014, 10:04 p.m., Àlex Fiestas wrote: > > I would move this perhaps to plasma-workspace since this slave is really > > important for nowadays usage of the desktop (android phones etc). Well, there's important kio's as well in kio-extras. Question is, is it useful outside the work

Re: Split kde-baseapps?

On Mon, Nov 10, 2014 at 4:46 PM, David Faure wrote: > If it's useful to share it, let's share it, I have no objection to that. How should this be done? Can I just copy things around and post a RR or should be this moved around by one of our git curators to preserve history? David E. Narvaez

Re: Review Request 119567: Also find catalog files that are symlinks

> On Aug. 2, 2014, 1:25 a.m., Luigi Toscano wrote: > > src/xslt.cpp, line 426 > > > > > > Uhm, but that's strange. Just to be sure, do I understand it correctly > > that every file in customization is sym-linked

Re: Review Request 121095: FrameworkIntegration: Add KTextToHTML emoticons support to FrameworkIntegrationPlugin

> On nov. 11, 2014, 2:39 après-midi, Albert Astals Cid wrote: > > I sincerely think this is a *VERY BAD* idea. I don't want my app behaving > > differently depending if a third party misterious compononent that is not > > documented anywhere is installed or not. If you have a dependency, well p

Re: Review Request 121095: FrameworkIntegration: Add KTextToHTML emoticons support to FrameworkIntegrationPlugin

--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/121095/#review70302 --- Since this seems quite easy to test, I would like automated te

Re: Review Request 121090: Move kio-mtp to kio-extras

--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/121090/#review70301 --- I would move this perhaps to plasma-workspace since this slave

Re: Review Request 121109: Use CMAKE_INSTALL_FULL_PLUGINDIR

--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/121109/ --- (Updated Nov. 12, 2014, 8:11 p.m.) Status -- This change has been ma

Re: Review Request 121109: Use CMAKE_INSTALL_FULL_PLUGINDIR

--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/121109/#review70295 --- Ship it! Thanks! - Ivan Čukić On Nov. 12, 2014, 7:43 p.m.,

Review Request 121109: Use CMAKE_INSTALL_FULL_PLUGINDIR

--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/121109/ --- Review request for KDE Frameworks and Ivan Čukić. Bugs: 340900 http:/

Re: Review Request 120760: Fix bug #340212: incorrect soft-tabs alignment after beginning-of-line

--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/120760/#review70293 --- Ship it! I think the changes makes sense and the code looks o

Re: Review Request 120760: Fix bug #340212: incorrect soft-tabs alignment after beginning-of-line

--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/120760/ --- (Updated nov. 12, 2014, 7:06 p.m.) Review request for Kate, KDE Framework

Re: Review Request 120760: Fix bug #340212: incorrect soft-tabs alignment after beginning-of-line

Ping? On Thu, Oct 23, 2014 at 10:09 AM, Michael Hansen wrote: >This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/120760/ > Review request for KDE Frameworks and Christoph Cullmann. > By Michael Hansen. > *Repository: * ktexteditor > Descriptio

Re: Review Request 121076: Do not require KF5::Su on Windows

--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/121076/#review70280 --- Ship it! Ship It! - Pedro Francisco On Nov. 8, 2014, 10:14

Re: Review Request 121095: FrameworkIntegration: Add KTextToHTML emoticons support to FrameworkIntegrationPlugin

> On nov. 11, 2014, 2:39 p.m., Albert Astals Cid wrote: > > I sincerely think this is a *VERY BAD* idea. I don't want my app behaving > > differently depending if a third party misterious compononent that is not > > documented anywhere is installed or not. If you have a dependency, well put >

Re: Review Request 121095: FrameworkIntegration: Add KTextToHTML emoticons support to FrameworkIntegrationPlugin

> On Nov. 11, 2014, 2:39 p.m., Albert Astals Cid wrote: > > I sincerely think this is a *VERY BAD* idea. I don't want my app behaving > > differently depending if a third party misterious compononent that is not > > documented anywhere is installed or not. If you have a dependency, well put >

Re: Review Request 121094: KCoreAddons: add KTextToHTML class for plaintext -> HTML conversion

> On Nov. 11, 2014, 12:22 p.m., Laurent Montel wrote: > > src/lib/text/ktexttohtml.cpp, line 380 > > > > > > same here There's no semantic difference between x++ and ++x here, I don't see the point of requestin