[Konversation-devel] [Bug 131954] open log in kate/favorite editing program
Eike Hein
hein at kde.org
Mon Apr 20 16:43:56 CEST 2009
https://bugs.kde.org/show_bug.cgi?id=131954
--- Comment #6 from Eike Hein <hein kde org> 2009-04-20 16:43:54 ---
Patch comments:
- Please don't use tabs for indentation - the patch contains a whole section
that does nothing except change space to tabs. Konvi uses spaces for
indentation.
- Rather than use KRun::run(), it would probably be better to just use the
regular KRun constructor, since you can then pass it useful parameters like
isLocalFile=true and showProgressInfo=false to improve speed and appearance.
Note that KRun auto-deletes itself when it's done, so there's no need to handle
that.
- "Use system text editor to view logs." would be a nicer/clearer option label
than "Use external log viewer".
--
Configure bugmail: https://bugs.kde.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
More information about the Konversation-devel
mailing list