Python Containment tutorial?

[alan moore]

Hi,
In the api documentation for PyKDE on the plasma containment page 
(http://api.kde.org/pykde-4.2-api/plasma/Plasma.Containment.html), it 
says "See techbase.kde.org for a tutorial on writing Containments using 
this class".

I'm interested in created a containment plasmoid using Python, and I'm 
wondering if anyone can point me to the URL of this tutorial.  There is 
no link on the API docs page and I haven't found a containment tutorial 
for Python on techbase.

Thanks.
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel

Re: scrolling text window

[Aaron J. Seigo]

On August 31, 2009, Daniel Duris wrote:
> I am developing a plasmoid displaying various logs in JS and I need
> this to make the flow of text automatically.

Plasma::TextBrowser?

-- 
Aaron J. Seigo
humru othro a kohnu se
GPG Fingerprint: 8B8B 2209 0C6F 7C47 B1EA  EE75 D6B7 2EB1 A7F1 DB43

KDE core developer sponsored by Qt Developent Framework


signature.asc
Description: This is a digitally signed message part.
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel

scrolling text window

[Daniel Duris]

Is there a psosibility to create not a LinearLayout, but layout with
automatically scrolling text window such as console-like (e.g. text is
pushed to the top, when screen full...)?

I am developing a plasmoid displaying various logs in JS and I need
this to make the flow of text automatically.

Thanks,

Daniel
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel

Re: problems with executable dataengine (and others) in Javascript

[Daniel Duris]

got it working, thanks.

On Mon, Aug 31, 2009 at 5:30 PM, Aaron J. Seigo wrote:
> On August 31, 2009, Daniel Duris wrote:
>> in my app. addSource() doesn't work (error is shown - can't start
>> script), setData - no syntax is known for JS
>
> connectSource, not add source :) so:
>
> dataengine("exec").connectSource("/bin/ls", plasmoid)
>
> --
> Aaron J. Seigo
> humru othro a kohnu se
> GPG Fingerprint: 8B8B 2209 0C6F 7C47 B1EA  EE75 D6B7 2EB1 A7F1 DB43
>
> KDE core developer sponsored by Qt Developent Framework
>
> ___
> Plasma-devel mailing list
> Plasma-devel@kde.org
> https://mail.kde.org/mailman/listinfo/plasma-devel
>
>
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel

Re: Review Request: Custom native widgets for Plasma::ComboBox, Plasma::LineEdit and Plasma::TextEdit

[Michal Dutkiewicz]

---
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/1094/
---

(Updated 2009-08-31 16:26:07.434735)


Review request for Plasma, Aaron Seigo and Marco Martin.


Changes
---

Include LineEdit and TextEdit too.


Summary (updated)
---

This patch adds possibility to use custom combo box widgets (sub classed from 
KComboBox) instead of plain KComboBox and similar possibilities for text and 
line edits.


Diffs (updated)
-

  /trunk/KDE/kdelibs/plasma/widgets/combobox.h 1016281 
  /trunk/KDE/kdelibs/plasma/widgets/combobox.cpp 1016281 
  /trunk/KDE/kdelibs/plasma/widgets/lineedit.h 1016281 
  /trunk/KDE/kdelibs/plasma/widgets/lineedit.cpp 1016281 
  /trunk/KDE/kdelibs/plasma/widgets/textedit.h 1016281 
  /trunk/KDE/kdelibs/plasma/widgets/textedit.cpp 1016281 

Diff: http://reviewboard.kde.org/r/1094/diff


Testing
---

Compiles and should work, but not tested.


Screenshots
---

Plasma themed KHistoryComboBox in Run Command applet
  http://reviewboard.kde.org/r/1094/s/151/


Thanks,

Michal

___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel

Re: Qalculate plasmoid moved to kdereview

[Burkhard Lück]

Am Montag 31 August 2009 16:35:52 schrieb Matteo Agostinelli:
> Hello list,
>
> first of all a disclaimer: this is my first contribution to KDE, so
> bear with me if I made any mistakes. I moved my Qalculate! plasmoid to
> kdereview aiming for an inclusion in kdeplasma-addons.
>
> Basically it is an extension of the existing calculator plasmoid,
> using the quite powerful Qalculate! library
> (http://qalculate.sourceforge.net/reference). Among the benefits:
> - unit conversion
> - symbolic expression evaluation
> - basic matrix and vector operations
>
> Since I am not an experienced KDE developer (nor a C++ guru, although
> I have some experience) I would really appreciate some feedback, to
> improve the quality of the applet and the code. 

Launching the Settings of qualculate i see a lot of 
"Settings(I18N_EXCESS_ARGUMENTS_SUPPLIED)" caused by broken i18n calls like
"i18nc("Evaluation", "Evaluation Settings", m_applet->name())"

The applet does not respect the locale settings for decimal separators (',' = 
comma in a german locale) in the input line.

Same problem as with kdeedu/step and libqalculate, see
https://bugs.kde.org/show_bug.cgi?id=181598

Does the applet support translated unit names in conversion?
Afaik libcalculate does not :-(

-- 
Burkhard Lück
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel

Re: Review Request: Keyboard navigation support for Tasks applet

[Marco Martin]

---
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/1482/#review2230
---

Ship it!


seems good

- Marco


On 2009-08-31 16:03:46, Michal Dutkiewicz wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> http://reviewboard.kde.org/r/1482/
> ---
> 
> (Updated 2009-08-31 16:03:46)
> 
> 
> Review request for Plasma, Aaron Seigo and Marco Martin.
> 
> 
> Summary
> ---
> 
> This patch adds support for [Shift + ] Tab navigation (and Enter / Return to 
> activate item) in Tasks plasmoid.
> In order to work it needs fixing of panel focusing issue (as discussed on 
> mailing list), I could apply mine method for temporary use if needed.
> 
> 
> Diffs
> -
> 
>   /trunk/KDE/kdebase/workspace/plasma/applets/tasks/abstracttaskitem.h 
> 1017535 
>   /trunk/KDE/kdebase/workspace/plasma/applets/tasks/abstracttaskitem.cpp 
> 1017535 
>   /trunk/KDE/kdebase/workspace/plasma/applets/tasks/taskgroupitem.h 1017535 
>   /trunk/KDE/kdebase/workspace/plasma/applets/tasks/taskgroupitem.cpp 1017535 
>   /trunk/KDE/kdebase/workspace/plasma/applets/tasks/windowtaskitem.h 1017535 
>   /trunk/KDE/kdebase/workspace/plasma/applets/tasks/windowtaskitem.cpp 
> 1017535 
> 
> Diff: http://reviewboard.kde.org/r/1482/diff
> 
> 
> Testing
> ---
> 
> Tested in plasmoidviwer and works (groups may need additional testing).
> 
> 
> Thanks,
> 
> Michal
> 
>

___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel

Re: Review Request: Keyboard navigation support for Tasks applet

[Michal Dutkiewicz]

---
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/1482/
---

(Updated 2009-08-31 16:03:46.974286)


Review request for Plasma, Aaron Seigo and Marco Martin.


Changes
---

Use available methods instead querying layout directly and possible 
optimization when there is only one task.


Summary
---

This patch adds support for [Shift + ] Tab navigation (and Enter / Return to 
activate item) in Tasks plasmoid.
In order to work it needs fixing of panel focusing issue (as discussed on 
mailing list), I could apply mine method for temporary use if needed.


Diffs (updated)
-

  /trunk/KDE/kdebase/workspace/plasma/applets/tasks/abstracttaskitem.h 1017535 
  /trunk/KDE/kdebase/workspace/plasma/applets/tasks/abstracttaskitem.cpp 
1017535 
  /trunk/KDE/kdebase/workspace/plasma/applets/tasks/taskgroupitem.h 1017535 
  /trunk/KDE/kdebase/workspace/plasma/applets/tasks/taskgroupitem.cpp 1017535 
  /trunk/KDE/kdebase/workspace/plasma/applets/tasks/windowtaskitem.h 1017535 
  /trunk/KDE/kdebase/workspace/plasma/applets/tasks/windowtaskitem.cpp 1017535 

Diff: http://reviewboard.kde.org/r/1482/diff


Testing
---

Tested in plasmoidviwer and works (groups may need additional testing).


Thanks,

Michal

___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel

Re: problems with executable dataengine (and others) in Javascript

[Aaron J. Seigo]

On August 31, 2009, Daniel Duris wrote:
> in my app. addSource() doesn't work (error is shown - can't start
> script), setData - no syntax is known for JS

connectSource, not add source :) so:

dataengine("exec").connectSource("/bin/ls", plasmoid)

-- 
Aaron J. Seigo
humru othro a kohnu se
GPG Fingerprint: 8B8B 2209 0C6F 7C47 B1EA  EE75 D6B7 2EB1 A7F1 DB43

KDE core developer sponsored by Qt Developent Framework


signature.asc
Description: This is a digitally signed message part.
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel

Re: Calling KDirLister from plasma widget causes no files to be found.

[David Hubner]

On Sunday 30 August 2009 08:53:45 Aaron J. Seigo wrote:
> On August 29, 2009, David Hubner wrote:
> > On Saturday 29 August 2009 18:37:27 Aaron J. Seigo wrote:
> > > On August 29, 2009, Aaron J. Seigo wrote:
> > > > KUniqueApplication vs KApplication
> > >
> > > the easy way to test this would be to change KApplication to
> > > KUniqueApplication in plasmoidviewer and see if that triggers the bug.
> >
> > Nope, plasmoidviewer still works. with #include  changed to
> > #include  and KApplication app; changed to
> > KUniqueApplication app;
> 
> great. so that's one more thing we can scratch off the list.
> 
> my next guess is a statically allocated object in plasma-desktop somewhere... 
> i'll take a glance through.
> 
> 

Looked through it today for a couple of hours, i can't see any static object 
that would effect KIO. I am hoping you have better luck :)

Thanks
-- 
David Hubner
IRC: Kirsan
MSN: hubn...@hubnerd.org
ICQ: 24308559
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel

problems with executable dataengine (and others) in Javascript

[Daniel Duris]

hello,

i am trying to develop a plasmoid in JS. However, I am unable to find
any sources of executable dataengine (or rss etc.). in
plasmaengineexplorer i can add commands to executable engine, but not
in my app. addSource() doesn't work (error is shown - can't start
script), setData - no syntax is known for JS

how can one use executable dataengine (and others) in JS?

Thanks,

Daniel
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel

Qalculate plasmoid moved to kdereview

[Matteo Agostinelli]

Hello list,

first of all a disclaimer: this is my first contribution to KDE, so
bear with me if I made any mistakes. I moved my Qalculate! plasmoid to
kdereview aiming for an inclusion in kdeplasma-addons.

Basically it is an extension of the existing calculator plasmoid,
using the quite powerful Qalculate! library
(http://qalculate.sourceforge.net/reference). Among the benefits:
- unit conversion
- symbolic expression evaluation
- basic matrix and vector operations

Since I am not an experienced KDE developer (nor a C++ guru, although
I have some experience) I would really appreciate some feedback, to
improve the quality of the applet and the code. Should I also post a
review request on http://reviewboard.kde.org?

Best regards,
Matteo
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel

Re: Review Request: big revamp of Device Notifier

[Giulio Camuffo]

> On 2009-08-29 21:00:29, Jacopo De Simoi wrote:
> > /trunk/KDE/kdebase/workspace/plasma/applets/devicenotifier/devicenotifier.cpp,
> >  line 204
> > 
> >
> > We probably need to change this;
> > Either show every time %i actions for this device and show the action 
> > items in this case as well, 
> > or make the action items appear only if there are more than 1 action. 
> > I would strongly suggest the second solution
> 
>  wrote:
> I know it. It is simply unuseful to show the action when you already know 
> what it is, but it was far in my priorities list :)
> 
>  wrote:
> now that I think about it, in my patch the KCapacityBar hides the 
> subtitle, so it would still be needed to show the action item. maybe i should 
> check if the subtitle is visible and in the case call directly the action.
> 
>  wrote:
> The KCapacityBar will eventually need to be shown only on hover, so the 
> proposed behavior should take into account this fact as well
>

yes, but the subtitle too is shown on hover only. So the problem remains. 
We could make the items bigger and paint the subtitle along with the 
KCapacityBar, as in the original patch for the capacity bar, but aesthetically 
I dislike it very much, because they become fat, or keep showing the item for a 
single action.

>Either show every time %i actions for this device and show the action items in 
>this case as well
I don't understand what you mean with this phrase. I'm not a native english 
speaker and some times I have some problems :)


- Giulio


---
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/1370/#review2179
---


On 2009-08-21 19:26:43, Giulio Camuffo wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> http://reviewboard.kde.org/r/1370/
> ---
> 
> (Updated 2009-08-21 19:26:43)
> 
> 
> Review request for Plasma.
> 
> 
> Summary
> ---
> 
> This is a patch that modifies quite heavily the behaviour of the Device 
> Notifier.
> It comes from here: 
> http://kde-look.org/content/show.php/Device+Manager?content=106051
> It can show the not removable devices too, it can mount them automatically or 
> with a click, since the "eject" button is a "mount" button when the volume is 
> umounted. So that guy on the dot will be ok.
> It can hide some items in the same way as Dolphin's places (hide item/ show 
> all).
> Finally, it shows the various opening actions under the device instead of 
> calling that xp-ish window.
> 
> 
> Diffs
> -
> 
>   
> /trunk/KDE/kdebase/workspace/plasma/applets/devicenotifier/devicespaceinfodelegate.cpp
>  1013960 
>   
> /trunk/KDE/kdebase/workspace/plasma/applets/devicenotifier/devicespaceinfodelegate.h
>  1013960 
>   
> /trunk/KDE/kdebase/workspace/plasma/applets/devicenotifier/devicenotifier.cpp 
> 1013960 
>   /trunk/KDE/kdebase/workspace/plasma/applets/devicenotifier/devicenotifier.h 
> 1013960 
>   
> /trunk/KDE/kdebase/workspace/plasma/applets/devicenotifier/configurationpage.ui
>  PRE-CREATION 
>   /trunk/KDE/kdebase/workspace/plasma/applets/devicenotifier/CMakeLists.txt 
> 1013960 
>   /trunk/KDE/kdebase/workspace/plasma/applets/devicenotifier/notifierdialog.h 
> 1013960 
>   
> /trunk/KDE/kdebase/workspace/plasma/applets/devicenotifier/notifierdialog.cpp 
> 1013960 
>   /trunk/KDE/kdebase/workspace/plasma/applets/devicenotifier/notifierview.h 
> 1013960 
>   /trunk/KDE/kdebase/workspace/plasma/applets/devicenotifier/notifierview.cpp 
> 1013960 
> 
> Diff: http://reviewboard.kde.org/r/1370/diff
> 
> 
> Testing
> ---
> 
> I'm using it every day since I released 0.1 on Kde-look. I tried all the 
> options on my pc and they work. Some people on kde-look posted some comments 
> about some problems, but it seems to me they are very particular cases, so in 
> my opinion it is quite stable to go in trunk, but anyway review it! :)
> 
> 
> Screenshots
> ---
> 
> screen
>   http://reviewboard.kde.org/r/1370/s/183/
> 
> 
> Thanks,
> 
> Giulio
> 
>

___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel

Re: Review Request: Keyboard navigation support for Tasks applet

[Michal Dutkiewicz]

---
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/1482/
---

(Updated 2009-08-31 14:04:45.714405)


Review request for Plasma, Aaron Seigo and Marco Martin.


Changes
---

Ported wheelEvent() to use the same logic for changing active task (hope that 
it is right approach).
By the way, why wheelEvent() is protected slot here?


Summary
---

This patch adds support for [Shift + ] Tab navigation (and Enter / Return to 
activate item) in Tasks plasmoid.
In order to work it needs fixing of panel focusing issue (as discussed on 
mailing list), I could apply mine method for temporary use if needed.


Diffs (updated)
-

  /trunk/KDE/kdebase/workspace/plasma/applets/tasks/abstracttaskitem.h 1017535 
  /trunk/KDE/kdebase/workspace/plasma/applets/tasks/abstracttaskitem.cpp 
1017535 
  /trunk/KDE/kdebase/workspace/plasma/applets/tasks/taskgroupitem.h 1017535 
  /trunk/KDE/kdebase/workspace/plasma/applets/tasks/taskgroupitem.cpp 1017535 
  /trunk/KDE/kdebase/workspace/plasma/applets/tasks/windowtaskitem.h 1017535 
  /trunk/KDE/kdebase/workspace/plasma/applets/tasks/windowtaskitem.cpp 1017535 

Diff: http://reviewboard.kde.org/r/1482/diff


Testing
---

Tested in plasmoidviwer and works (groups may need additional testing).


Thanks,

Michal

___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel

Re: Review Request: Quicklaunch: Fix config UI where mouse wheeling on slider does not change spinbox value

[Beat Wolf]

> On 2009-08-02 23:52:28, Aaron Seigo wrote:
> > good catch :)
> 
> Shafqat Bhuiyan wrote:
> Thanks :)
> 
> It would be great if you could commit this one as well :)

this patch is commited and should be marked like that


- Beat


---
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/1221/#review1886
---


On 2009-08-02 23:42:14, Shafqat Bhuiyan wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> http://reviewboard.kde.org/r/1221/
> ---
> 
> (Updated 2009-08-02 23:42:14)
> 
> 
> Review request for Plasma.
> 
> 
> Summary
> ---
> 
> This changes the slider signals which connect to the spinboxes from 
> sliderMoved(int) to valueChanged(int) in the quicklaunch configuration 
> window. This fixes the problem where mouse wheeling on the slider does not 
> change the spinbox.
> 
> This should be backported to 4.3
> 
> 
> Diffs
> -
> 
>   
> /trunk/KDE/kdebase/workspace/plasma/applets/quicklaunch/quicklaunchConfig.ui 
> 1004927 
> 
> Diff: http://reviewboard.kde.org/r/1221/diff
> 
> 
> Testing
> ---
> 
> Tested on trunk
> 
> 
> Thanks,
> 
> Shafqat
> 
>

___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel

Re: Review Request: FIXME part of quicklaunch

[sujith h]

On 8/31/09, Aaron J. Seigo  wrote:
> On August 31, 2009, Beat Wolf wrote:
>> > On 2009-05-07 10:17:53, Aaron Seigo wrote:
>> > > another nice bug fix for 4.3 :)
>>
>> this patch seems to be commited, how can one mark a reviewrequest as
>> shipped?
>
> the reporter needs to do it (or someone with admin rights, which i have
> though
> i can't make other people admins; have to ask David S. for that)

Its fault from my side. And the request is marked as submitted. Thanks.

I am marking the features/bugs that I have committed to trunk as submitted.


Sujith H

-- 
സുജിത് ഹരിദാസന്
Bangalore
http://fci.wikia.com/wiki/Anti-DRM-Campaign
 http://sujithh.info
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel

Re: Review Request: Add tooltips to the SystemTray show/icon hidden icons button

[Darío Andrés]

On Mon, Aug 31, 2009 at 5:27 AM, Beat Wolf wrote:
>
>
>> On 2009-05-05 15:57:15, Aaron Seigo wrote:
>> > this will have to wait for 4.4 now due to the string freeze, but that will 
>> > also give us time to track down the tooltip issue? :)
>
> did anybody commit this? so the bugreport can be closed
>
>

I haven't checked this against 4.4 to see if the bug I described was
fixed, or if it can be workarounded. I'm going to do it later.
Regards

> - Beat
>
>
> ---
> This is an automatically generated e-mail. To reply, visit:
> http://reviewboard.kde.org/r/630/#review1069
> ---
>
>
> On 2009-04-25 15:31:01, Darío Andrés wrote:
>>
>> ---
>> This is an automatically generated e-mail. To reply, visit:
>> http://reviewboard.kde.org/r/630/
>> ---
>>
>> (Updated 2009-04-25 15:31:01)
>>
>>
>> Review request for Plasma.
>>
>>
>> Summary
>> ---
>>
>> This adds a tooltip to the SystemTray icon which hides or shows the hidden 
>> icons. It may need better texts.
>> Addresses bug 183953
>>
>>
>> This addresses bug 183953.
>>     https://bugs.kde.org/show_bug.cgi?id=183953
>>
>>
>> Diffs
>> -
>>
>>   
>> svn://anonsvn.kde.org/home/kde/trunk/KDE/kdebase/workspace/plasma/applets/systemtray/ui/taskarea.cpp
>>  958965
>>
>> Diff: http://reviewboard.kde.org/r/630/diff
>>
>>
>> Testing
>> ---
>>
>>
>> Thanks,
>>
>> Darío
>>
>>
>
>
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel

Re: Review Request: big revamp of Device Notifier

[Jacopo De Simoi]

---
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/1370/#review2213
---


I previously overlooked this; the mounting procedure should belong to 
notifierdialog, as the unmounting. Please remove all the slot/signal stuff. 
It's not necessary and makes the code more complicated. 


/trunk/KDE/kdebase/workspace/plasma/applets/devicenotifier/devicenotifier.cpp


I do not see why this method belongs here. 
Unmounting is done internally by the notifierdialog, so mounting should be 
taken care of similarly. 
Please move this piece of code where it belongs (see the other note)



/trunk/KDE/kdebase/workspace/plasma/applets/devicenotifier/notifierdialog.cpp


Please move the mounting method here, there's no need for the signal/slot 
dance.
Moreover this makes it possible to trigger the error dialog in case 
something goes wrong (see right above)


Thanks

- Jacopo


On 2009-08-21 19:26:43, Giulio Camuffo wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> http://reviewboard.kde.org/r/1370/
> ---
> 
> (Updated 2009-08-21 19:26:43)
> 
> 
> Review request for Plasma.
> 
> 
> Summary
> ---
> 
> This is a patch that modifies quite heavily the behaviour of the Device 
> Notifier.
> It comes from here: 
> http://kde-look.org/content/show.php/Device+Manager?content=106051
> It can show the not removable devices too, it can mount them automatically or 
> with a click, since the "eject" button is a "mount" button when the volume is 
> umounted. So that guy on the dot will be ok.
> It can hide some items in the same way as Dolphin's places (hide item/ show 
> all).
> Finally, it shows the various opening actions under the device instead of 
> calling that xp-ish window.
> 
> 
> Diffs
> -
> 
>   
> /trunk/KDE/kdebase/workspace/plasma/applets/devicenotifier/devicespaceinfodelegate.cpp
>  1013960 
>   
> /trunk/KDE/kdebase/workspace/plasma/applets/devicenotifier/devicespaceinfodelegate.h
>  1013960 
>   
> /trunk/KDE/kdebase/workspace/plasma/applets/devicenotifier/devicenotifier.cpp 
> 1013960 
>   /trunk/KDE/kdebase/workspace/plasma/applets/devicenotifier/devicenotifier.h 
> 1013960 
>   
> /trunk/KDE/kdebase/workspace/plasma/applets/devicenotifier/configurationpage.ui
>  PRE-CREATION 
>   /trunk/KDE/kdebase/workspace/plasma/applets/devicenotifier/CMakeLists.txt 
> 1013960 
>   /trunk/KDE/kdebase/workspace/plasma/applets/devicenotifier/notifierdialog.h 
> 1013960 
>   
> /trunk/KDE/kdebase/workspace/plasma/applets/devicenotifier/notifierdialog.cpp 
> 1013960 
>   /trunk/KDE/kdebase/workspace/plasma/applets/devicenotifier/notifierview.h 
> 1013960 
>   /trunk/KDE/kdebase/workspace/plasma/applets/devicenotifier/notifierview.cpp 
> 1013960 
> 
> Diff: http://reviewboard.kde.org/r/1370/diff
> 
> 
> Testing
> ---
> 
> I'm using it every day since I released 0.1 on Kde-look. I tried all the 
> options on my pc and they work. Some people on kde-look posted some comments 
> about some problems, but it seems to me they are very particular cases, so in 
> my opinion it is quite stable to go in trunk, but anyway review it! :)
> 
> 
> Screenshots
> ---
> 
> screen
>   http://reviewboard.kde.org/r/1370/s/183/
> 
> 
> Thanks,
> 
> Giulio
> 
>

___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel

Re: Review Request: Patch to add free space indicator into device notifier plasmoid

[Giulio Camuffo]

In data martedì 25 agosto 2009 10:50:58, Kevin Ottens ha scritto:
: > On Saturday 22 August 2009 16:02:56 Jacopo De Simoi wrote:
> > I see two solutions:
> > - Once the device is made accessible trigger an update of the free disk
> > space every (say) 10 seconds - Make the KCapacityBar appear on hover and
> > trigger an update of the free disk space on hover;
> >
> > I personally like the second possibility more than the first one,
> > Best
>
> +10
>
> That's what we're doing in the KFilePlaces* component because the last
> thing we want is an app constantly polling your disk (and preventing them
> from spinning down) just for that bit of information.
>
> Regards.

Isn't it possible to use INotify, or a similar thing to update the free space 
only 
when something changed?

Giulio.
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel

Re: Review Request: Keyboard navigation support for Tasks applet

[Michal Dutkiewicz]

> On 2009-08-31 08:58:13, Marco Martin wrote:
> > /trunk/KDE/kdebase/workspace/plasma/applets/tasks/taskgroupitem.cpp, line 
> > 696
> > 
> >
> > should be circular and logic for this is already in the wheel event, 
> > you should use that ()then wheelevent calling focusPrevNex (don't really 
> > like the name btw)

This is Qt built in method, I'll port wheel event to use it (using built in 
handling of Tab is best way I think ;-)).
But there is one problem, in case of wheel event we can simply activate window 
but in case of [Shift + ] Tab it is not so easy, especially before fixing focus 
for panels, so I'll stick with Enter / Return to activate when using keyboard.


- Michal


---
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/1482/#review2208
---


On 2009-08-30 20:46:50, Michal Dutkiewicz wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> http://reviewboard.kde.org/r/1482/
> ---
> 
> (Updated 2009-08-30 20:46:50)
> 
> 
> Review request for Plasma, Aaron Seigo and Marco Martin.
> 
> 
> Summary
> ---
> 
> This patch adds support for [Shift + ] Tab navigation (and Enter / Return to 
> activate item) in Tasks plasmoid.
> In order to work it needs fixing of panel focusing issue (as discussed on 
> mailing list), I could apply mine method for temporary use if needed.
> 
> 
> Diffs
> -
> 
>   /trunk/KDE/kdebase/workspace/plasma/applets/tasks/taskgroupitem.h 1017471 
>   /trunk/KDE/kdebase/workspace/plasma/applets/tasks/taskgroupitem.cpp 1017471 
>   /trunk/KDE/kdebase/workspace/plasma/applets/tasks/abstracttaskitem.h 
> 1017471 
>   /trunk/KDE/kdebase/workspace/plasma/applets/tasks/abstracttaskitem.cpp 
> 1017471 
>   /trunk/KDE/kdebase/workspace/plasma/applets/tasks/windowtaskitem.h 1017471 
>   /trunk/KDE/kdebase/workspace/plasma/applets/tasks/windowtaskitem.cpp 
> 1017471 
> 
> Diff: http://reviewboard.kde.org/r/1482/diff
> 
> 
> Testing
> ---
> 
> Tested in plasmoidviwer and works (groups may need additional testing).
> 
> 
> Thanks,
> 
> Michal
> 
>

___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel

Re: Review Request: FIXME part of quicklaunch

[Aaron J. Seigo]

On August 31, 2009, Beat Wolf wrote:
> > On 2009-05-07 10:17:53, Aaron Seigo wrote:
> > > another nice bug fix for 4.3 :)
>
> this patch seems to be commited, how can one mark a reviewrequest as
> shipped?

the reporter needs to do it (or someone with admin rights, which i have though 
i can't make other people admins; have to ask David S. for that)


-- 
Aaron J. Seigo
humru othro a kohnu se
GPG Fingerprint: 8B8B 2209 0C6F 7C47 B1EA  EE75 D6B7 2EB1 A7F1 DB43

KDE core developer sponsored by Qt Software


signature.asc
Description: This is a digitally signed message part.
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel

Re: Review Request: Keyboard navigation support for Tasks applet

[Marco Martin]

---
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/1482/#review2208
---



/trunk/KDE/kdebase/workspace/plasma/applets/tasks/taskgroupitem.cpp


should be circular and logic for this is already in the wheel event, you 
should use that ()then wheelevent calling focusPrevNex (don't really like the 
name btw)


- Marco


On 2009-08-30 20:46:50, Michal Dutkiewicz wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> http://reviewboard.kde.org/r/1482/
> ---
> 
> (Updated 2009-08-30 20:46:50)
> 
> 
> Review request for Plasma, Aaron Seigo and Marco Martin.
> 
> 
> Summary
> ---
> 
> This patch adds support for [Shift + ] Tab navigation (and Enter / Return to 
> activate item) in Tasks plasmoid.
> In order to work it needs fixing of panel focusing issue (as discussed on 
> mailing list), I could apply mine method for temporary use if needed.
> 
> 
> Diffs
> -
> 
>   /trunk/KDE/kdebase/workspace/plasma/applets/tasks/taskgroupitem.h 1017471 
>   /trunk/KDE/kdebase/workspace/plasma/applets/tasks/taskgroupitem.cpp 1017471 
>   /trunk/KDE/kdebase/workspace/plasma/applets/tasks/abstracttaskitem.h 
> 1017471 
>   /trunk/KDE/kdebase/workspace/plasma/applets/tasks/abstracttaskitem.cpp 
> 1017471 
>   /trunk/KDE/kdebase/workspace/plasma/applets/tasks/windowtaskitem.h 1017471 
>   /trunk/KDE/kdebase/workspace/plasma/applets/tasks/windowtaskitem.cpp 
> 1017471 
> 
> Diff: http://reviewboard.kde.org/r/1482/diff
> 
> 
> Testing
> ---
> 
> Tested in plasmoidviwer and works (groups may need additional testing).
> 
> 
> Thanks,
> 
> Michal
> 
>

___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel

Re: Review Request: FIXME part of quicklaunch

[Beat Wolf]

> On 2009-05-07 10:17:53, Aaron Seigo wrote:
> > another nice bug fix for 4.3 :)

this patch seems to be commited, how can one mark a reviewrequest as shipped?


- Beat


---
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/649/#review1087
---


On 2009-05-07 08:56:57, Sujith  H wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> http://reviewboard.kde.org/r/649/
> ---
> 
> (Updated 2009-05-07 08:56:57)
> 
> 
> Review request for Plasma and Aaron Seigo.
> 
> 
> Summary
> ---
> 
> This patch is regarding the FIXME part of the quicklaunchApplet.cpp. I had 
> added one more method named dropApp. Added the processDrop as per Aaron 
> seigo. 
> 
> 
> Diffs
> -
> 
>   trunk/KDE/kdebase/workspace/plasma/applets/quicklaunch/quicklaunchApplet.h 
> 964135 
>   
> trunk/KDE/kdebase/workspace/plasma/applets/quicklaunch/quicklaunchApplet.cpp 
> 964135 
> 
> Diff: http://reviewboard.kde.org/r/649/diff
> 
> 
> Testing
> ---
> 
> Done basic testing and it works.
> 
> 
> Thanks,
> 
> Sujith
> 
>

___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel

Re: Review Request: Add tooltips to the SystemTray show/icon hidden icons button

[Beat Wolf]

> On 2009-05-05 15:57:15, Aaron Seigo wrote:
> > this will have to wait for 4.4 now due to the string freeze, but that will 
> > also give us time to track down the tooltip issue? :)

did anybody commit this? so the bugreport can be closed


- Beat


---
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/630/#review1069
---


On 2009-04-25 15:31:01, Darío Andrés wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> http://reviewboard.kde.org/r/630/
> ---
> 
> (Updated 2009-04-25 15:31:01)
> 
> 
> Review request for Plasma.
> 
> 
> Summary
> ---
> 
> This adds a tooltip to the SystemTray icon which hides or shows the hidden 
> icons. It may need better texts.
> Addresses bug 183953
> 
> 
> This addresses bug 183953.
> https://bugs.kde.org/show_bug.cgi?id=183953
> 
> 
> Diffs
> -
> 
>   
> svn://anonsvn.kde.org/home/kde/trunk/KDE/kdebase/workspace/plasma/applets/systemtray/ui/taskarea.cpp
>  958965 
> 
> Diff: http://reviewboard.kde.org/r/630/diff
> 
> 
> Testing
> ---
> 
> 
> Thanks,
> 
> Darío
> 
>

___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel