Bug 157513 - settings: provide configurable notifications
Summary: settings: provide configurable notifications
Status: RESOLVED FIXED
Alias: None
Product: ktorrent
Classification: Applications
Component: general (show other bugs)
Version: unspecified
Platform: openSUSE Linux
: NOR wishlist
Target Milestone: ---
Assignee: Joris Guisson
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-02-09 13:03 UTC by Maciej Pilichowski
Modified: 2008-02-25 10:38 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed In:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Maciej Pilichowski 2008-02-09 13:03:13 UTC
Version:            (using KDE 3.5.8)
Installed from:    SuSE RPMs

Currently notifications are hardcoded -- it would be nice to have them configurable.
Comment 1 Joris Guisson 2008-02-09 16:50:11 UTC
What do you want to configure about them ? 

Btw, you have been pretty busy, it was quite a surprise to suddenly see around 15 new bugreports when I check my e-mail 
Comment 2 Maciej Pilichowski 2008-02-09 17:05:13 UTC
> What do you want to configure about them ? 

Me personally -- sounds. I.e. I would like to add small sound notification for download complete (when torrent goes from download to upload) + sound notifications for errors.
 
> Btw, you have been pretty busy, it was quite a surprise to suddenly see
> around 15 new bugreports when I check my e-mail 

11 more and I will celebrate my 1500th report :-)) Sorry for troubling you though, however since I like your program and use it... ;-D
 
Comment 3 Joris Guisson 2008-02-25 10:38:43 UTC
SVN commit 779018 by guisson:

Use KNotify system for notifications instead of own hardcoded one with KPassivePopup. Also make this configurable using the standard configure notifications dialog.

BUG: 157513


 M  +2 -1      CMakeLists.txt  
 M  +7 -0      gui.cpp  
 M  +1 -1      gui.h  
 A             ktorrent.notifyrc  
 M  +17 -15    trayicon.cpp  
 M  +1 -0      trayicon.h  


WebSVN link: http://websvn.kde.org/?view=rev&revision=779018