Bug 211725 - Konversation has a bad default configuration
Summary: Konversation has a bad default configuration
Status: RESOLVED INTENTIONAL
Alias: None
Product: konversation
Classification: Applications
Component: general (show other bugs)
Version: 1.2
Platform: Mandriva RPMs Linux
: NOR wishlist
Target Milestone: ---
Assignee: Konversation Developers
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2009-10-25 01:24 UTC by rapsys
Modified: 2009-10-27 15:46 UTC (History)
3 users (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments
konversation-1.2 default config fix version following kdelibs-4.3+ api (2.90 KB, patch)
2009-10-25 01:25 UTC, rapsys
Details
Removed the hide on startup (2.61 KB, patch)
2009-10-26 22:51 UTC, rapsys
Details
Log file explaining the difficulty in changing the default log file path. (8.73 KB, text/plain)
2009-10-27 10:26 UTC, Eike Hein
Details

Note You need to log in before you can comment on or make changes to this bug.
Description rapsys 2009-10-25 01:24:07 UTC
Version:           1.2 (using KDE 4.3.2)
Compiler:          gcc version 4.4.1 (GCC) Target: i586-manbo-linux-gnu
Configuré avec: ../configure --prefix=/usr --libexecdir=/usr/lib --with-slibdir=/lib --with-bugurl=https://qa.mandriva.com/ --mandir=/usr/share/man --infodir=/usr/share/info --enable-checking=release --enable-languages=c,c++,ada,fortran,objc,obj-c++,java --build=i586-manbo-linux-gnu --host=i586-manbo-linux-gnu --with-cpu=generic --with-system-zlib --enable-threads=posix --enable-shared --enable-objc-gc --enable-long-long --enable-__cxa_atexit --disable-libunwind-exceptions --enable-clocale=gnu --enable-java-awt=gtk --with-java-home=/usr/lib/jvm/java-1.5.0-gcj-1.5.0.0/jre --with-ecj-jar=/usr/share/java/eclipse-ecj.jar --enable-gtk-cairo --disable-libjava-multilib --enable-ssp --disable-libssp --disable-werror --with-ppl --with-cloog --with-python-dir=/lib/python2.6/site-packages
OS:                Linux
Installed from:    Mandriva RPMs

Konversation has bad default configuration and path

Plz apply next attachement patch that fix it.

I did it to get a good configuration by default in mandriva :
https://qa.mandriva.com/show_bug.cgi?id=37329
Comment 1 rapsys 2009-10-25 01:25:07 UTC
Created attachment 37808 [details]
konversation-1.2 default config fix version following kdelibs-4.3+ api
Comment 2 Eike Hein 2009-10-26 00:19:48 UTC
The DCC destination path was already fixed in SVN on October 15th (back to the previous default of $HOME/dccrecv).

The other configuration options you changed, like enabling the tray icon by default, are the way they are intentionally (and have been this way for years) and will not be changed.
Comment 3 Eike Hein 2009-10-26 00:43:33 UTC
Hm, after having a second look, using KGlobalSettings::downloadPath() for the DCC folder might not be a bad idea though. I'll ask our DCC maintainer.
Comment 4 rapsys 2009-10-26 22:51:50 UTC
Created attachment 37864 [details]
Removed the hide on startup

Plz keep the other default config change.

- Left tab is required to allow having more than 20 channels and keep a usable ihm.
- Activate trayicon like does kopete, that way end user will know there is this possibility
- Skip useless event and similar are mandatory to avoid beeing notified by anoying bot or similar noise on non-unix-guru channels
- downloadPath is required to finish sending files in non xdg dir
- log is required to finish send logs in home dir
Comment 5 Nicolas L. 2009-10-27 00:41:02 UTC
i just  reopen because of the new patch
Comment 6 Eike Hein 2009-10-27 08:47:39 UTC
> - Left tab is required to allow having more than 20 channels and keep a usable ihm.

Most users don't start out joining 20 channels, and with relatively few channels, the treelist version of the tab bar is a waste of screen space. Our defaults are designed to cater to new users, not power users. We expect power users to be able to change the position of the tab bar on their own.


> - Activate trayicon like does kopete, that way end user will know there is this possibility


We strongly object to applications polluting the system tray by default. Tray icons should be opt-in, not opt-out. We also don't believe in enabling features by default merely to advertise their existence; that is bad software development practice.


> - Skip useless event and similar are mandatory to avoid beeing notified by
anoying bot or similar noise on non-unix-guru channels

That join/part/nick events are "useless" is your personal opinion. It is not ours. We do not believe that an IRC client should hide them from users by default, and indeed no popular IRC client I'm aware of does.


> - downloadPath is required to finish sending files in non xdg dir

This has been discussed with our DCC maintainer -- he likes it and will make the change.


> - log is required to finish send logs in home dir

I can't see any compelling reasons to hide the logfiles in a hidden directory structure where they're difficult to access. IRC users tend to want to grep their logfiles frequently; the default log path reflects that.


I understand that you prefer to have your IRC client configured differently, and that's fine, that's why we have these options. Defaults are not for everyone. But please do realize that in chosing the defaults, we have a greater audience to consider than just you, or ourselves for that matter.
Comment 7 Eike Hein 2009-10-27 08:52:53 UTC
(An addendum to the join/part/nick events thing: Note that these are classified as "Channel events" and won't light up the tab labels by default, for example. Thus there's no need to "to avoid beeing notified by anoying bot or similar noise" because there in fact is no such notification by default. Your change would disable them from deing displayed in the text views by default, which is a definitive no-go.)
Comment 8 Eike Hein 2009-10-27 10:26:15 UTC
Created attachment 37906 [details]
Log file explaining the difficulty in changing the default log file path.
Comment 9 Eike Hein 2009-10-27 15:46:17 UTC
SVN commit 1041186 by hein:

Use XDG download directory as default destination folder for
incoming DCC file transfers as suggested by rapsys and ack'ed
by buscher.
CCBUG:211725


 M  +1 -1      konversation.kcfg  


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