[Kommander-devel] plugin notices

[Alexander Fischer]

Hello,


I have four requests.
First is, that there should be an addition in relation to file 
src/kommander/working/plugintemplate/plugin.cpp, line 19

addWidget( "%{APPNAME}", "Custom", i18n("Kommander %{APPNAME} plugin."), 
new QIconSet(KGlobal::iconLoader()->loadIcon("%{APPNAMELC}", 
KIcon::NoGroup, KIcon::SizeMedium)) );

I would say there should be either a comment like 

/*If you don't plan to use an 
unique self-constructed icon to your plugin, change the first parameter in 
function loadIcon to a KDE default icon of your choice. Note that this icon 
must be unique inside kmdr-editor.*/

or this fact should be added to the documentation.

Another point is, the plugin template is only available as new project while I
run kdevelop as C and C++ IDE ("kdevelop --profile CandCppIDE") and is not 
available while running kdevelop as KDE IDE (kdevelop --profile KDECppIDE).
This was confusing to me because kommander is a KDE app and of course the 
plugin should be likewise.

The next point is, a fact I never realized before playing with my OpenWithDlg 
plugin, that all dialogs using standard kde/qt functions 
(input_color, input_directory, input_...) are only available in english, even 
if there are locales installed and the kmdr-editor is using a language different 
to english. I don't know if this is a real problem but I wondered. 
This is especially amazing as kmdr-executor makes use of  the i18n function 
inside kmdr-scripts.

And at the end I have to say I was hasty releasing my OpenWithDlg plugin. 
A version 0.2 is available here 
http://www.ordnungsdienst.net/plugin-openwithdlg-0.2.tar.bz2 .
I added two functions getSelection() and getExecSelection()
The first opens the OpenWith-... dialog and just returns the user input 
without executing the selected application. This might useful if someone likes
to save a default application inside the kmdr-script which is different 
from the system default. GetExecSelection returns the last user selection 
because the execute function doesn't do this.

It would be nice, if someone with to much time could check the code, but I think it is ok.
I hope this is the last modification the next time :-)


cu alex