How is plasma active handling copy/paste in QML?

2012-11-26 Thread Mark
Hi, For the next step in my project i need to have copy/paste support. On the C++ side this is done in KIO where Dolphin wraps around it a little and uses the konqueror version. I'm guessing plasma active has file management functionalities and can handle copy/cut/paste stuff on the QML side, rig

Re: Review Request: Plasmate: on SigningDialog::validateParams() replace QRegExp with KPIMUtils::isValidSimpleAddress

2012-11-26 Thread Diego Casella
> On Nov. 26, 2012, 1:43 p.m., Diego Casella wrote: > > plasmate/CMakeLists.txt, line 102 > > > > > > I'm wondering if we really need to add a kdepimutils dependency for a > > simple email validation... Thoughts ab

Re: Review Request: Plasmate: on SigningDialog::validateParams() replace QRegExp with KPIMUtils::isValidSimpleAddress

2012-11-26 Thread Diego Casella
> On Nov. 26, 2012, 8:53 p.m., Diego Casella wrote: > > plasmate/CMakeLists.txt, line 102 > > > > > > >>I'm wondering if we really need to add a kdepimutils dependency for a > > simple email validation... Thoughts

Re: Review Request: Plasmate: on SigningDialog::validateParams() replace QRegExp with KPIMUtils::isValidSimpleAddress

2012-11-26 Thread Diego Casella
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/107470/#review22572 --- plasmate/CMakeLists.txt

Re: Review Request: make the DeviceNotifier Plasmoid scale with high-res screens

2012-11-26 Thread Marco Martin
> On Nov. 26, 2012, 9:35 a.m., Marco Martin wrote: > > This is really an issue that needs addressing, thanks for taking time for > > it. > > > > i'm very in favor of removing all those hardcoded sizes from the code. > > > > however i see two problems with that approach > > > > 1) icons are cl

Review Request: Save and restore position of widgets in vertically-oriented FlowGroup

2012-11-26 Thread Dan Vrátil
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/107477/ --- Review request for Plasma. Description --- This fixes a bug when widg

Re: Homerun sources and Plasma runners

2012-11-26 Thread Aurélien Gâteau
Le jeudi 22 novembre 2012 13:29:06 Aaron J. Seigo a écrit : > > Homerun AbstractSource class is not a model. It inherits from QObject and > > provide virtual methods to create models. A source must implement at > > least: > > "QAbstractItemModel *createModelFromConfigGroup(const KConfigGroup > > &

Re: Review Request: make the DeviceNotifier Plasmoid scale with high-res screens

2012-11-26 Thread Michael Zanetti
> On Nov. 26, 2012, 9:35 a.m., Marco Martin wrote: > > This is really an issue that needs addressing, thanks for taking time for > > it. > > > > i'm very in favor of removing all those hardcoded sizes from the code. > > > > however i see two problems with that approach > > > > 1) icons are cl

Re: Review Request: populate the settings of the rssnow-qml plasmoid

2012-11-26 Thread Giorgos Tsiapaliokas
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/106582/ --- (Updated Nov. 26, 2012, 2:45 p.m.) Review request for Plasma. Changes --

Re: Review Request: Plasmate: on SigningDialog::validateParams() replace QRegExp with KPIMUtils::isValidSimpleAddress

2012-11-26 Thread Antonis Tsiapaliokas
> On Nov. 26, 2012, 1:43 p.m., Diego Casella wrote: > > plasmate/CMakeLists.txt, line 102 > > > > > > I'm wondering if we really need to add a kdepimutils dependency for a > > simple email validation... Thoughts ab

Re: Review Request: Plasmate: on SigningDialog::validateParams() replace QRegExp with KPIMUtils::isValidSimpleAddress

2012-11-26 Thread Giorgos Tsiapaliokas
> On Nov. 26, 2012, 1:43 p.m., Diego Casella wrote: > > plasmate/CMakeLists.txt, line 102 > > > > > > I'm wondering if we really need to add a kdepimutils dependency for a > > simple email validation... Thoughts ab

Re: Review Request: make the DeviceNotifier Plasmoid scale with high-res screens

2012-11-26 Thread Jacopo De Simoi
> On Nov. 26, 2012, 9:35 a.m., Marco Martin wrote: > > This is really an issue that needs addressing, thanks for taking time for > > it. > > > > i'm very in favor of removing all those hardcoded sizes from the code. > > > > however i see two problems with that approach > > > > 1) icons are cl

Re: Review Request: Plasmate: on SigningDialog::validateParams() replace QRegExp with KPIMUtils::isValidSimpleAddress

2012-11-26 Thread Diego Casella
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/107470/#review22561 --- plasmate/CMakeLists.txt

Re: Review Request: make the DeviceNotifier Plasmoid scale with high-res screens

2012-11-26 Thread Marco Martin
> On Nov. 26, 2012, 9:35 a.m., Marco Martin wrote: > > This is really an issue that needs addressing, thanks for taking time for > > it. > > > > i'm very in favor of removing all those hardcoded sizes from the code. > > > > however i see two problems with that approach > > > > 1) icons are cl

Re: Review Request: make the DeviceNotifier Plasmoid scale with high-res screens

2012-11-26 Thread Michael Zanetti
> On Nov. 26, 2012, 9:35 a.m., Marco Martin wrote: > > This is really an issue that needs addressing, thanks for taking time for > > it. > > > > i'm very in favor of removing all those hardcoded sizes from the code. > > > > however i see two problems with that approach > > > > 1) icons are cl

Re: Review Request: make the DeviceNotifier Plasmoid scale with high-res screens

2012-11-26 Thread Marco Martin
> On Nov. 26, 2012, 9:35 a.m., Marco Martin wrote: > > This is really an issue that needs addressing, thanks for taking time for > > it. > > > > i'm very in favor of removing all those hardcoded sizes from the code. > > > > however i see two problems with that approach > > > > 1) icons are cl

Review Request: Plasmate: on SigningDialog::validateParams() replace QRegExp with KPIMUtils::isValidSimpleAddress

2012-11-26 Thread Antonis Tsiapaliokas
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/107470/ --- Review request for Plasma. Description --- On this patch i am replace

Re: Review Request: make the DeviceNotifier Plasmoid scale with high-res screens

2012-11-26 Thread Marco Martin
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/107428/#review22548 --- This is really an issue that needs addressing, thanks for takin