Re: Review Request: Use Plasma::ScrollWidget in the widget explorer

2010-08-04 Thread Aurélien Gâteau
--- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/4676/ --- (Updated 2010-08-04 21:02:11.815343) Review request for Plasma. Changes --

Re: Review Request: Adding a chronometer to the fifteenPuzzle plasmoid

2010-08-04 Thread Marco Martin
> On 2010-08-04 18:16:18, Aaron Seigo wrote: > > nice idea! > > > > instead of a popup dialog, how about putting a QGraphicsWidget over the > > Applet with the message? perhaps even Plasma::Applet::showMessage? that function is there exactly for those kinds of use cases, so yea it should be u

Re: Review Request: Fixing positions on the fifteenPuzzle to make it solvable

2010-08-04 Thread Alex Raymond
> On 2010-08-04 18:58:29, Aaron Seigo wrote: > > looks good :) > > > > (you don't have an svn account, correct? will you be continuing to work on > > paches for KDE? if so, we could get you your own svn account and you could > > help maintain the 15 puzzle plasmoid and others. if you're just t

Re: Review Request: Fixing positions on the fifteenPuzzle to make it solvable

2010-08-04 Thread Aaron Seigo
--- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/4867/#review6800 --- Ship it! looks good :) (you don't have an svn account, correct? will

Re: Review Request: Supporting theme colors on the fifteenPuzzle plasmoid

2010-08-04 Thread Aaron Seigo
> On 2010-08-04 18:33:46, Aaron Seigo wrote: > > is that really a problem? if you choose a custom colour .. you choose a > > custom colour. > > Alex Raymond wrote: > But the problem is that if you manually select the theme color from the > theme A, which is red, and then change to theme B,

Re: Review Request: Fixing positions on the fifteenPuzzle to make it solvable

2010-08-04 Thread Alex Raymond
--- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/4867/ --- (Updated 2010-08-04 18:41:05.064228) Review request for Plasma and Aaron Seigo.

Re: Review Request: Supporting theme colors on the fifteenPuzzle plasmoid

2010-08-04 Thread Alex Raymond
> On 2010-08-04 18:33:46, Aaron Seigo wrote: > > is that really a problem? if you choose a custom colour .. you choose a > > custom colour. But the problem is that if you manually select the theme color from the theme A, which is red, and then change to theme B, which is blue. It will keep op

Re: Review Request: Fixing positions on the fifteenPuzzle to make it solvable

2010-08-04 Thread Aaron Seigo
--- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/4867/#review6797 --- Ship it! one small issue below in terms of coding style, but this loo

Re: Review Request: Supporting theme colors on the fifteenPuzzle plasmoid

2010-08-04 Thread Aaron Seigo
--- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/4904/#review6796 --- is that really a problem? if you choose a custom colour .. you choose

Re: Review Request: Adding a chronometer to the fifteenPuzzle plasmoid

2010-08-04 Thread Aaron Seigo
--- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/4870/#review6795 --- nice idea! instead of a popup dialog, how about putting a QGraphicsWi

Re: Review Request: share dataengine review improvements

2010-08-04 Thread Artur de Souza (MoRpHeUz)
--- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/4903/#review6794 --- Ship it! Great patch. Thanks for the feedback! - Artur On 2010-08-

Review Request: Supporting theme colors on the fifteenPuzzle plasmoid

2010-08-04 Thread Alex Raymond
--- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/4904/ --- Review request for Plasma, Aaron Seigo and Tomaz Canabrava. Summary --- Thi

Re: Review Request: Adding a chronometer to the fifteenPuzzle plasmoid

2010-08-04 Thread Alex Raymond
--- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/4870/ --- (Updated 2010-08-04 18:02:52.817973) Review request for Plasma and Aaron Seigo.

Review Request: share dataengine review improvements

2010-08-04 Thread Aaron Seigo
--- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/4903/ --- Review request for Plasma and Artur de Souza (MoRpHeUz). Summary --- this p

Re: Review Request: Fixing positions on the fifteenPuzzle to make it solvable

2010-08-04 Thread Alex Raymond
--- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/4867/ --- (Updated 2010-08-04 17:20:03.485141) Review request for Plasma and Aaron Seigo.

Re: Review Request: Enable the boundingRect to grow along with the board on the fifteenPuzzle plasmoid.

2010-08-04 Thread Alex Raymond
--- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/4869/ --- (Updated 2010-08-04 15:45:51.073400) Review request for Plasma and Aaron Seigo.

Re: [Nepomuk] Activities and desktop events - Results from the meeting

2010-08-04 Thread Sebastian Trüg
On 08/03/2010 09:17 PM, Ivan Čukić wrote: >> standardQuery( ResourcesForActivityQuery, ResourceTerm(activity) ) >> && >> standardQuery( ResourcesForApplicationQuery, ResourceTerm(app) ) >> && >> standardQuery( MostImportantResourcesQuery ) > > Ok, works for me. very good. >> how so? and which ap