Re: Review Request: Unified ItemBackground in new Device Notifier

2009-10-06 Thread Jacopo De Simoi
--- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/1790/#review2562 --- Ship it! I'm finally convinced, we need to fix some issues here and

Review Request: Plasma D-Bus Interface: Setting wallpaper image

2009-10-06 Thread Ivan Cukic
--- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/1798/ --- Review request for Plasma. Summary --- There are many users who want the

Plasma-Model/View-Nepomuk

2009-10-06 Thread PierreL
Hi, We have a plasmoid that reads Nepomuk (as RDF store) and creates a model and view. Well this model is created when the plasmoid is started. The thing is now, with this plasmoid already added in a previous KDE session, when KDE starts, it seems, plasma is started before Nepomuk service or

Re: Plasma-Model/View-Nepomuk

2009-10-06 Thread Ivan Čukić
Hi, what happens when you just restart plasma (kquitapp plasma-desktop; sleep 2; plasma-desktop) - does it load the data then? How do you access nepomuk? Via its classes or via d-bus? session, when KDE starts, it seems, plasma is started before Nepomuk You can check whether this is the case

Review Request: kineticscrolling, with pixel based properties

2009-10-06 Thread Marco Martin
--- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/1799/ --- Review request for Plasma, igorto and Adenilson Cavalcanti. Summary ---

Re: Review Request: Unified ItemBackground in new Device Notifier

2009-10-06 Thread Aaron Seigo
On 2009-10-06 10:16:00, Jacopo De Simoi wrote: /trunk/kdereview/plasma/applets/devicenotifier-refactor/deviceitem.cpp, line 390 http://reviewboard.kde.org/r/1790/diff/3/?file=12370#file12370line390 Now that the left(hehe)action stays activated when !isCollapsed() we should

Calculator runner

2009-10-06 Thread Matteo Agostinelli
Hello, as suggested by Aaron in a previous discussion, I am having a look at the calculator runner with the purpose of extending the capabilities of the runner by using libqalculate when it is available, otherwise falling back to the currently used 'qscript' method. So here are my questions

Re: New devicenotifier moved to kdereview

2009-10-06 Thread Kevin Ottens
On Tuesday 6 October 2009 21:49:36 Giulio Camuffo wrote: or do you mean it doesn't show your / or /home? unfortunately the dataengines don't manage volumes handles by fstab, but only by hal That'd be surprising. Those volumes are also reported by HAL (and then Solid), the data engine

Re: Plasma-Model/View-Nepomuk

2009-10-06 Thread Aaron J. Seigo
On October 6, 2009, PierreL wrote: The thing is now, with this plasmoid already added in a previous KDE session, when KDE starts, it seems, plasma is started before Nepomuk service or maybe I'm wrong, but the plasmoid's model is empty (not populated with any Nepomuk data). either

Re: Calculator runner

2009-10-06 Thread Aaron J. Seigo
On October 6, 2009, Aleix Pol wrote: You can use KAlgebra code which is inside kdeedu. instead of libqalculate? besides creating an odd dependency, what would the benefits be? -- Aaron J. Seigo humru othro a kohnu se GPG Fingerprint: 8B8B 2209 0C6F 7C47 B1EA EE75 D6B7 2EB1 A7F1 DB43 KDE

Re: Review Request: Plasma D-Bus Interface: Setting wallpaper image

2009-10-06 Thread Aaron Seigo
--- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/1798/#review2568 --- shouldn't the wallpaper export its own dbus interface, and the path

KineticScrolling strategies

2009-10-06 Thread Marco Martin
Hi all, the Plasma::KineticScrolling private class had a quite a painful history. it was wrote accessing directly the scrollwidget internal widgeet, then migrated to a property based system using percentages as scroll values. now it has still some problems, the horizontal and vertical scroll

Re: New devicenotifier moved to kdereview

2009-10-06 Thread Kevin Ottens
On Tuesday 6 October 2009 20:56:53 Aaron J. Seigo wrote: On October 6, 2009, Kevin Ottens wrote: probably filter them out somehow. There's an ignore hint on those devices which it probably honors by default, it's probably just a matter of changing the behavior of the engine to report them

Re: New devicenotifier moved to kdereview

2009-10-06 Thread Giulio Camuffo
In data martedì 06 ottobre 2009 18:16:23, Aaron J. Seigo ha scritto: : On October 6, 2009, Giulio Camuffo wrote: or do you mean it doesn't show your / or /home? unfortunately the dataengines don't manage volumes handles by fstab, but only by hal it doesn't show either of my disk

Re: Review Request: Plasma D-Bus Interface: Setting wallpaper image

2009-10-06 Thread Ivan Cukic
On 2009-10-06 19:05:00, Aaron Seigo wrote: shouldn't the wallpaper export its own dbus interface, and the path to that dbus object be dependent on the containment id? e.g. something like Containments/1/Wallpaper? then instead of a plugin-specific hack, we could have per-plugin

Re: Review Request: Unified ItemBackground in new Device Notifier

2009-10-06 Thread Giulio Camuffo
On 2009-10-06 10:16:00, Jacopo De Simoi wrote: /trunk/kdereview/plasma/applets/devicenotifier-refactor/deviceitem.cpp, line 152 http://reviewboard.kde.org/r/1790/diff/3/?file=12370#file12370line152 Here we should make sure to hide() the description Label; wrote: playing

Re: KineticScrolling strategies

2009-10-06 Thread Marco Martin
On Tuesday 06 October 2009, Artur Souza (MoRpHeUz) wrote: Hello :) On Tuesday 06 October 2009, 16:15 Marco Martin wrote: the Plasma::KineticScrolling private class had a quite a painful history. it was wrote accessing directly the scrollwidget internal widgeet, then migrated to a property

Re: Calculator runner

2009-10-06 Thread Matteo Agostinelli
In data martedì 6 ottobre 2009 20:57:00, Aleix Pol ha scritto: : Well, depends a lot on how odd it is, you could install it within kdeedu the same way we do with the kalgebra plasmoid. There's no odd dependency crossing here. Benefits, you would be using some maintained code inside KDE.

Re: New devicenotifier moved to kdereview

2009-10-06 Thread Aaron J. Seigo
On October 6, 2009, Giulio Camuffo wrote: In data martedì 06 ottobre 2009 18:16:23, Aaron J. Seigo ha scritto: : On October 6, 2009, Giulio Camuffo wrote: : or do you mean it doesn't show your / or /home? unfortunately the dataengines don't manage volumes handles by fstab, but only by

Re: Calculator runner

2009-10-06 Thread David Baron
On Tuesday 06 October 2009 19:09:02 Matteo Agostinelli wrote: Hello, as suggested by Aaron in a previous discussion, I am having a look at the calculator runner with the purpose of extending the capabilities of the runner by using libqalculate when it is available... There already is such

Re: New devicenotifier moved to kdereview

2009-10-06 Thread Aaron J. Seigo
On October 6, 2009, Giulio Camuffo wrote: in my opinion the applet should show all the devices, fixed or not, fstab or not. i tried with the solidengine, but it doesn't show the other devices too. i think anyway a better approach would be to add an action file for them, and add a property

Re: New devicenotifier moved to kdereview

2009-10-06 Thread Jacopo De Simoi
On Tuesday 06 October 2009 21:58:08 Aaron J. Seigo wrote: i think anyway a better approach would be to add an action file for them, and add a property in the DeviceItem to set if they are (un)mountable it would prevent any special-case code in the notifier widget, yes. If we have no

Re: Review Request: Plasma D-Bus Interface: Setting wallpaper image

2009-10-06 Thread Aaron Seigo
On 2009-10-06 19:05:00, Aaron Seigo wrote: shouldn't the wallpaper export its own dbus interface, and the path to that dbus object be dependent on the containment id? e.g. something like Containments/1/Wallpaper? then instead of a plugin-specific hack, we could have per-plugin

Re: Calculator runner

2009-10-06 Thread Aaron J. Seigo
On October 6, 2009, Aleix Pol wrote: Well, depends a lot on how odd it is, you could install it within kdeedu the same way we do with the kalgebra plasmoid. There's no odd dependency crossing here. the problem would be that we would then have two calculation plugins: one in kdebase (because

Re: New devicenotifier moved to kdereview

2009-10-06 Thread Jacopo De Simoi
On Tuesday 06 October 2009 22:17:02 Aaron J. Seigo wrote: so much for simplicity :/ I'm confused; I can read in test-predicate-openinwindow.desktop: X-KDE-Solid-Predicate=[ [ StorageVolume.ignored == false AND [ StorageVolume.usage == 'FileSystem' OR StorageVolume.usage == 'Encrypted' ] ] OR

Re: New devicenotifier moved to kdereview

2009-10-06 Thread Kevin Ottens
On Tuesday 6 October 2009 22:17:02 Aaron J. Seigo wrote: On October 6, 2009, Giulio Camuffo wrote: in my opinion the applet should show all the devices, fixed or not, fstab or not. i tried with the solidengine, but it doesn't show the other devices too. i think anyway a better approach

Re: Calculator runner

2009-10-06 Thread Aaron J. Seigo
On October 6, 2009, David Baron wrote: On Tuesday 06 October 2009 19:09:02 Matteo Agostinelli wrote: Hello, as suggested by Aaron in a previous discussion, I am having a look at the calculator runner with the purpose of extending the capabilities of the runner by using libqalculate

Re: New devicenotifier moved to kdereview

2009-10-06 Thread Aaron J. Seigo
On October 6, 2009, Jacopo De Simoi wrote: On Tuesday 06 October 2009 21:58:08 Aaron J. Seigo wrote: i think anyway a better approach would be to add an action file for them, and add a property in the DeviceItem to set if they are (un)mountable it would prevent any special-case

Re: New devicenotifier moved to kdereview

2009-10-06 Thread Aaron J. Seigo
On October 6, 2009, Jacopo De Simoi wrote: On Tuesday 06 October 2009 22:17:02 Aaron J. Seigo wrote: so much for simplicity :/ I'm confused; I can read in test-predicate-openinwindow.desktop: X-KDE-Solid-Predicate=[ [ StorageVolume.ignored == false AND [ StorageVolume.usage ==

Re: Calculator runner

2009-10-06 Thread David Baron
On Tuesday 06 October 2009 22:36:52 Aaron J. Seigo wrote: On October 6, 2009, David Baron wrote: On Tuesday 06 October 2009 19:09:02 Matteo Agostinelli wrote: Hello, as suggested by Aaron in a previous discussion, I am having a look at the calculator runner with the purpose of

Review Request: Plasma::WindowEffects functions to render window thumbnails

2009-10-06 Thread Marco Martin
--- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/1800/ --- Review request for Plasma. Summary --- this is the tiny part for the

Re: Calculator runner

2009-10-06 Thread Aleix Pol
On Tue, Oct 6, 2009 at 10:27 PM, Aaron J. Seigo ase...@kde.org wrote: On October 6, 2009, Aleix Pol wrote: Well, depends a lot on how odd it is, you could install it within kdeedu the same way we do with the kalgebra plasmoid. There's no odd dependency crossing here. the problem would

Panel autohide

2009-10-06 Thread Thomas Olsen
Hi We just had a talk on kde-linux[*] about auto hiding panels and I asked if anyone knew if it was possible to have a delay on showing panels? Is this implemented or planned to be? [*] http://lists.kde.org/?l=kde-linuxm=125486478602064w=2 -- Best Regards / Med venlig hilsen Thomas Olsen

Re: Review Request: Plasma::WindowEffects functions to render window thumbnails

2009-10-06 Thread Aaron Seigo
--- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/1800/#review2572 --- great to see this code moving into Plasma::WindowEffects. (maybe we