QML DataSource

2011-10-05 Thread Viranch Mehta
Hi all, I am facing a very annoying problem in QML plasmoids with the dataengines. The following does not seem to work ever for almost all dataengines: PlasmaCore.DataSource { id: dataSource engine: "someEngine" onSourceAdded: print (data[source]["some_valid_key"]; } The print statem

Re: Review Request: Workaround Taskbar ghost entries bug

2011-10-05 Thread Craig Drummond
On Wednesday 05 Oct 2011, Aaron J. Seigo wrote: > On Wednesday, October 5, 2011 20:08:55 Craig Drummond wrote: > > I don’t know if its related, but whilst working on IconTasks I noticed > > that items set to be deleted via deleteLater (in > > TaskGroupItem::itemRemoved and TaskManager::TaskItem) we

Re: Review Request: Workaround Taskbar ghost entries bug

2011-10-05 Thread Aaron J. Seigo
On Wednesday, October 5, 2011 20:08:55 Craig Drummond wrote: > I don’t know if its related, but whilst working on IconTasks I noticed that > items set to be deleted via deleteLater (in TaskGroupItem::itemRemoved and > TaskManager::TaskItem) were not being deleted until after the app > (plasmoidview

Re: Review Request: Workaround Taskbar ghost entries bug

2011-10-05 Thread Craig Drummond
I don’t know if its related, but whilst working on IconTasks I noticed that items set to be deleted via deleteLater (in TaskGroupItem::itemRemoved and TaskManager::TaskItem) were not being deleted until after the app (plasmoidviewer / plasma-desktop) had terminated. To trace this I put debug wh

Re: Review Request: Workaround Taskbar ghost entries bug

2011-10-05 Thread Commit Hook
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/102779/#review7115 --- This review has been submitted with commit 4fb69cc9bcda8974a220

Re: Review Request: Workaround Taskbar ghost entries bug

2011-10-05 Thread Commit Hook
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/102779/#review7114 --- This review has been submitted with commit 681e1ef4b5f0f7e66d33

Re: removing dependencies from kactivitymanagerd

2011-10-05 Thread Aaron J. Seigo
On Wednesday, October 5, 2011 12:49:58 Ivan Čukić wrote: > > i really recommend KDBusService. > > Ok, so that means this will be a brang of kamd for kf5? no, i don't think we need that. for now we can copy that one class into a frameworks/ folder and then when we port this to f5, we can git rid

Re: removing dependencies from kactivitymanagerd

2011-10-05 Thread Ivan Čukić
> i really recommend KDBusService. Ok, so that means this will be a brang of kamd for kf5? ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel

Re: Review Request: Simplify pixmap handling for XEmbed-based systray

2011-10-05 Thread Commit Hook
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/102770/#review7113 --- This review has been submitted with commit ae04d14bd49772257871

Re: Review Request: XTest is an optional dependency, so skip a test that requires it if it is not found

2011-10-05 Thread Commit Hook
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/102771/#review7111 --- This review has been submitted with commit ad68a0e4f51425400a37

Re: Review Request: fix the problem that the text below the selected wallpaper is barely readable if the window is inactive

2011-10-05 Thread Commit Hook
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/102772/#review7112 --- This review has been submitted with commit d97ddda3e5c801486797

Re: Review Request: fix the problem that the text below the selected wallpaper is barely readable if the window is inactive

2011-10-05 Thread Commit Hook
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/102772/#review7110 --- This review has been submitted with commit 3dece5943c07b1bb2a1b

Re: Review Request: Simplify pixmap handling for XEmbed-based systray

2011-10-05 Thread Aaron J. Seigo
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/102770/#review7109 --- Ship it! Ship It! - Aaron J. Seigo On Oct. 4, 2011, 3:31 p.m

Re: Review Request: Simplify pixmap handling for XEmbed-based systray

2011-10-05 Thread Aaron J. Seigo
> On Oct. 5, 2011, 1:58 a.m., Aaron J. Seigo wrote: > > thanks for the patch ... :) > > > > the coding style should follow the kdelibs style, but that's a minor thing. > > mostly just the one question regarding something noted in a code comment: > > Luboš Luňák wrote: > I can run the file

Re: Review Request: Workaround Taskbar ghost entries bug

2011-10-05 Thread Aaron J. Seigo
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/102779/#review7107 --- the check makes sense; still wish we knew *where* these odd task

Re: removing dependencies from kactivitymanagerd

2011-10-05 Thread Aaron J. Seigo
On Tuesday, October 4, 2011 18:29:33 Ivan Cukic wrote: > General question - do you want to create a fw5 branch or something like > that? > > Aaron > > > KUniqueApplication: this lives in kdeui ... just to provide a way to > > have only one instance of the app. ugh. in Frameworks there is libkdbus

Re: Replace kimpanel in kdeplasma-addons with a rewrite version

2011-10-05 Thread Aaron J. Seigo
On Wednesday, October 5, 2011 00:31:21 Weng Xuetian wrote: > 在 2011年10月4日 Tuesday 17:19:40,Aaron J. Seigo 写道: > For me that's what I exactly want to do in the futre: try to get them merge > if necessary. And I'm also a input method developer (named fcitx, which is > not a part of K

Review Request: Workaround Taskbar ghost entries bug

2011-10-05 Thread Alex Fiestas
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/102779/ --- Review request for Plasma and Aaron J. Seigo. Description --- Well, a

Re: Review Request: Simplify pixmap handling for XEmbed-based systray

2011-10-05 Thread Luboš Luňák
> On Oct. 5, 2011, 1:58 a.m., Aaron J. Seigo wrote: > > thanks for the patch ... :) > > > > the coding style should follow the kdelibs style, but that's a minor thing. > > mostly just the one question regarding something noted in a code comment: I can run the file through the astyle command be

Re: Review Request: Plasmate: Metadata is now able to detect the correct api

2011-10-05 Thread Aaron J. Seigo
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/102609/#review7104 --- Ship it! Ship It! - Aaron J. Seigo On Sept. 24, 2011, 12:28

Re: Review Request: XTest is an optional dependency, so skip a test that requires it if it is not found

2011-10-05 Thread Ralf Jung
> On Oct. 5, 2011, 1:53 a.m., Aaron J. Seigo wrote: > > Ship It! Could someone please do that for me? I do not have commit access. - Ralf --- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/1

Re: Review Request: fix the problem that the text below the selected wallpaper is barely readable if the window is inactive

2011-10-05 Thread Ralf Jung
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/102772/ --- (Updated Oct. 5, 2011, 8:55 a.m.) Review request for Plasma. Changes ---