showdesktop in kdereview, proposed move into extragear

[Loïc Marteau]

Bonsoir Anne-Marie ;o)

In fact i have the same problem (ShowDestop/minimize-all have been 
comming back to extragear)...

I see that the macro MINIMIZE_ONLY has been commited today to 
deactivated some stuff in the applet. It seems that it does not resolve 
the problem and Petri says that it is perhaps a plasma issue.

I can't commit my patch since i don't have svn access. Can you do it ?

Cheers !

Loic

Anne-Marie Mahfouf wrote:
> Hi Loïc,
>
> Did you try putting MinimizeAll in the panel? We are several people having a 
> huge CPU load when MinimizeAll is in the panel. So this issue is to tackle 
> because it's quite bad.
> I don't have kdereveiw to check ShowDesktop but I hope it doe snot do the 
> same.
>
> I agree with the change of names.
> Cheers,
>
> Anne-Marie
>
> On Friday 23 May 2008 06:46:33 pm Loïc Marteau wrote:
>   
>> Lubos Lunak wrote:
>>     
>>> On Saturday 03 of May 2008, okparanoid wrote:
>>>       
>>>> Davide Bettio wrote:
>>>>         
>>>>> Hi again,
>>>>>
>>>>> I just moved showdesktop plasma applet to kdereview.
>>>>> showdesktop is an applet that allows users to show plasmoids on desktop
>>>>> (like when you use CTRL+F12 shortcut).
>>>>>
>>>>> Bye,
>>>>> Davide Bettio.
>>>>>           
>>>> Hi !
>>>>
>>>> About this two applets, in an user point of view, i think that the name
>>>> of showdesktop is a little confusing.
>>>> Why not using showDashBoard instead which reflect more the action of the
>>>> applet ?
>>>> Same thing for the icon...
>>>>
>>>>
>>>>
>>>> The ideal for clarity will to rename minimize-all to showDesktop, and
>>>> showDesktop to showDashboard.
>>>>         
>>>  Especially given that minimize-all does not really minimize all. Not
>>> that "Show Desktop" is a better name, apparently, given all the user
>>> confusion about what it actually does.
>>>       
>> Hi devs !
>>
>> Here is my proposal patch to rename showDesktop to showDashboard and
>> minimize-all to showDesktop.
>>
>> I'm using the icon "user-desktop" of the old showDesktop plasmoid for
>> the new one (the minimize-all one is a litlle ugly) and make reference
>> to a new one "user-dashboard" (that i don't have make :op) for
>> showDashboard, in fact perhaps they can share the same, or an artist can
>> do one for showdashboard ?!
>>
>> m_icon = new Plasma::Icon(KIcon("user-dashboard"),QString(),this);
>>
>> Can this patch being apply ? Perhaps that initial authors have to be ok
>> with that too ?!
>>
>> I have to create a showdesktop.tmp directory because svn need to delete
>> the old one before.
>> If it is ok for apply the patch you can commit, then move
>> showdesktop.tmp to showdesktop in a second time and commit the mv.
>>
>> I will perhaps make an other little plasmoid to have a shortcut for
>> krunner. Thank for this apps it is very great and i wish that end-user
>> can find them easily, they are afraid by keyboard shortcut  ! It is
>> possible to add krunner directly in favorites apps by default for 4.1 ?!
>>
>> My personal opinion is that this two basics applets are very useful for
>> end user and in fact i think that the correct place for them is more
>> directly in kdebase...
>>
>> Best regards, sorry for my poor english.
>>
>> PS: I'm very excited about the incoming release of Kde. The work you
>> have made is sooo amazing and i'm quite sure that it will be a killer
>> Desktop that will make lot of proprietary Desktop user the switch ! I
>> want really thank all of you for that.
>>
>> Loic
>>     
>
>
>
> _______________________________________________
> Panel-devel mailing list
> Panel-devel at kde.org
> https://mail.kde.org/mailman/listinfo/panel-devel
>