Re: [PATCH] BUG 202473 no sound on emptying trash widget although sound is configured in systemsettings

2009-09-03 Thread 潘卫平(Peter Pan)
Aaron J. Seigo 写道:
 On September 1, 2009, 潘卫平(Peter Pan) wrote:
 How about the patch followed?
 
 the applet would need to follow the contents of trash:/ then, so when a file 
 is trashed in dolphin, konqueror or some other app it would become enabled.
 

That has been already done in trash applet using KDirLister.

Regards
-- 
潘卫平(Peter Pan)
Red Flag Software Co., Ltd
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


KDE/kdebase/workspace/plasma/tools/desktopthemedetails

2009-09-03 Thread Anne-Marie Mahfouf
SVN commit 1019435 by annma:

reflect new location of Desktop Theme settings.
However that makes 2 different locations for Desktop Themes in System Settings, 
could they be both under Appearance for example? Or shouldn't we ask the 
Usability Team about it?
ccmail=plasma-de...@kde.org


 M  +1 -1  desktopthemedetails.cpp  


--- 
trunk/KDE/kdebase/workspace/plasma/tools/desktopthemedetails/desktopthemedetails.cpp
 #1019434:1019435
@@ -464,7 +464,7 @@
 if (m_themeModel-indexOf(themeRoot) != -1) {
 m_theme-setCurrentIndex(m_themeModel-indexOf(themeRoot));
 //FIXME: should say Appearance Settings instead of Desktop Settings
-KMessageBox::information(this,i18n(To change your desktop theme to 
\%1\, open\n the desktop Appearance Settings and select \%2\ from the 
droplist.,m_theme-currentText(),m_theme-currentText() ), i18n(How to Change 
Desktop Theme), HowToChangeDesktopTheme);
+KMessageBox::information(this,i18n(To change your desktop theme to 
\%1\, get back to the General tab of SystemSettings in Appearance - Style 
and select \%2\ from the 
droplist.,m_theme-currentText(),m_theme-currentText() ), i18n(How to Change 
Desktop Theme), HowToChangeDesktopTheme);
 }
 resetThemeDetails();
 }
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: [PATCH] disable the fade effect from the slidingpopups effect

2009-09-03 Thread Lucas Murray
On Thu, Sep 3, 2009 at 7:36 PM, Marco Martinnotm...@gmail.com wrote:
 this patch adds a proxy to the fade effect that permits to set ignored
 windows, all the windows where slidingpopups is applied add themseves
 to this list

- Does not meet coding style (Check whitespace between parentheses and
end of lines as well as brace locations)
- Example comments have not been removed (E.g. // Example proxy code,
TODO: Use or remove)
- Typos in other comments
- kDebug() should be in zone 1212 (Although I think this is no longer
required in trunk it should still be added for consistency)
- Those debug comments probably shouldn't even be required
- FadeEffect::proxy() should not be const'ed on either side due to
recent proxy API changes at your request
- Code does not take into account when a window initially slides out
yet has been configured to not slide back in on close (Currently it
prevents the window from being faded as windows are only ever added to
the list, never removed)
- Memory leak: Windows are not removed from FadeEffect::ignoredWindows
when they are deleted

-- 

Lucas Murray :: http://www.undefinedfire.com
GPG Fingerprint: 0B88 499E 3F5B 1405 D952  258A AD90 B4F5 90B6 3534
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Review Request: Support for the plasma:/ protocol in urls from emails/web pages and the network:/ kioslave

2009-09-03 Thread Friedrich W. H. Kossebau


 On 2009-09-03 19:01:20, Rob Scheepmaker wrote:
  Cool, nice work. Indeed support for dragdropping those urls and 'opening' 
  them should still be added. I've added that to my TODO. I'm currently at 
  tokamak, but will be going away tomorrow morning and have some other stuff 
  to do as well, so I don't think I'll be able to do this here, but I'll make 
  some time next week.
  I'll create some function to the plasma-desktop dbus interface probably, 
  create some small helper program that invokes this, and of course add the 
  dragdrop support which is all quite trivial. This will all greatly help in 
  making the remote widgets feature more discoverable and accessable.

Great :) Waiting for next week or whatever your (and mine) time permits.
Perhaps you even do not need to write a special helper program then, and
exec=qdbus org.kde.plasma /Desktop openService %U
or similar (no Plasma Desktop running currently to check for real D-Bus 
addressing) should be sufficient.
Hm, but what to do with Plasma services which are rather designed for e.g. 
Amarok? No idea now.


- Friedrich W. H.


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


On 2009-09-03 18:23:03, Friedrich W. H. Kossebau wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 http://reviewboard.kde.org/r/1515/
 ---
 
 (Updated 2009-09-03 18:23:03)
 
 
 Review request for Plasma.
 
 
 Summary
 ---
 
 Hi!
 
 With commit #1019443 to kdebase/runtime/kioslave/network (done today) I added 
 a new entry for Plasma service to the DNSSD (zeroconf) backend for the 
 network:/ kioslave, which means that network:/ should now show a nice Plasma 
 icon for such services and have a link plasma:/hostname:port/name connected 
 to the entry.
 (Beware, you need to restart kded after updating your install, as the 
 kioslave is feeded by a kded module, which has the data in the binary (yes, 
 TODO :) )! Perhaps you even have to load the module manually, the automatic 
 load is reported to sometimes fail:
 qdbus org.kde.kded /kded loadModule networkwatcher).
 
 Now, the listing in network:/ is one thing, one also wants to deal with the 
 service item in Konqueror, e.g. click on it or drag'n'drop it to the Plasma 
 workspace. The same happens if the plasma:/ url is used in web pages or 
 emails (Son, here you can connect to my Dinner-is-ready plasmoid, Yours, 
 Mum), or isn't this supposed to be done?
 With KIO there is the need of a .protocol file which describes what the 
 plasma:/ protocol is about (see patch for prototype). AFAIK for such 
 protocols not starting a kioslave, but a helper program (helper=true), that 
 one needs to be defined here in the exec= line. So what would the helper 
 program be for plasma:/ urls? For Drag'nDrops this entry is ignored, BTW, and 
 just the url passed.
 
 
 Diffs
 -
 
   trunk/KDE/kdelibs/plasma/CMakeLists.txt 1019436 
   trunk/KDE/kdelibs/plasma/plasma.protocol PRE-CREATION 
 
 Diff: http://reviewboard.kde.org/r/1515/diff
 
 
 Testing
 ---
 
 
 Thanks,
 
 Friedrich W. H.
 


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


change background hits

2009-09-03 Thread sacha schutz
Hi,

What do you think about adding a default option in all plasmoid that allow to 
enable or disable the background ? 

Here you can see an example : 

without bkg: http://www.kde-look.org/CONTENT/content-pre1/98838-1.png
with bkg: http://www.kde-look.org/CONTENT/content-pre3/98838-3.png
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: change background hits

2009-09-03 Thread Aaron J. Seigo
On September 3, 2009, sacha schutz wrote:
 What do you think about adding a default option in all plasmoid that allow
  to enable or disable the background ?

this is already controllable by the 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 Development 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: change background hits

2009-09-03 Thread Aaron J. Seigo
On September 3, 2009, sacha schutz wrote:
 Le jeudi 3 septembre 2009 23:15:08, Aaron J. Seigo a écrit :
  On September 3, 2009, sacha schutz wrote:
   What do you think about adding a default option in all plasmoid that
   allow to enable or disable the background ?
 
  this is already controllable by the plasmoid.
 
 By the plasmoid, mean by the source code.  I mean Change the background by
  the user. 

consistency and not tweaking every last pixel on the user end is part of the 
theory.

  In the config pannel for example.
 This is an example : http://blog.nixternal.com/wp-
 content/uploads/2007/11/kde4fun.png 
 . In this screenshot , the desktop is
 ugly .

posting a screenshot of the 4.0 theme doesn't really make your point very well 
;)

 It would be cool if we can set plasmoid to transparent by clicking on
  a button in plasmoid config pannel..

you can: by setting the theme.

-- 
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 Development 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


applet dragdrop between panel and desktop

2009-09-03 Thread WuYongbo
Hi guys,

I am very curious the way plasma used to dragdrop applets between panel and
desktop:

1, drag applets from desktop to panel: user cannot drag applets directly but
with a handle, why ?
2, drag applets from panel to desktop: user cannot drag applets directly but
with a PanelAppletOverlay cover, why ?

why plasma does not use the *Drag and Drop* mechanism ?

BTW, what should I do if I want to drag applets from desktop to panel
directly ?

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