Summary: | open log in kate/favorite editing program | ||
---|---|---|---|
Product: | [Applications] konversation | Reporter: | Niek Beernink <n.beernink> |
Component: | general | Assignee: | Konversation Developers <konversation-devel> |
Status: | RESOLVED FIXED | ||
Severity: | wishlist | CC: | hein |
Priority: | NOR | ||
Version: | unspecified | ||
Target Milestone: | --- | ||
Platform: | Compiled Sources | ||
OS: | Linux | ||
Latest Commit: | Version Fixed In: | ||
Attachments: |
Adds external log viewer to preferences under behavior > general
Adds external log viewer to preferences under behavior > logging Try 3 of adding external log viewing :) |
Description
Niek Beernink
2006-08-06 14:23:46 UTC
I further suggest that a button that will start the external app be added to the current log tab. Created attachment 32628 [details]
Adds external log viewer to preferences under behavior > general
You should be able to use this patch to get the functionality you want. This is my first time dabbling with the konversation source code, but it seems like an easy enough addition.
Hi Mickey, thank you for your patch, but unfortunately Konversation 1.1 was the last feature release developed for KDE 3, and thus we cannot accept patches that add new features or strings to that development branch (it accepts fixes only). If you want your work to be included, I'd ask you to update it for the KDE 4 port, which you may currently find under branches/work/konvi-kde4 in SVN (and soon again in trunk/extragear/network). However, I wonder if a more user-friendly approach here would be to make the option a selection between "Built-in log viewer" and "My KDE text editor", and in the latter case, run the app associated with the text/plain MIME type via the respective kdelibs API. Created attachment 32939 [details]
Adds external log viewer to preferences under behavior > logging
This should add the option to the svn version. Should it be under logging? or under general?
Under Logging seems fine, thanks. 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". s/space /spaces / and no, the dot at the end wasn't supposed to be part of the suggested better option label, sorry. Actually not sure about the isLocalFile, though - I guess technically we do allow the user to pick a remote location as log file folder. Created attachment 32969 [details]
Try 3 of adding external log viewing :)
Here's try 3.
Thanks for the feedback on the other patch. I'm just getting started in working with KDE projects, so I'm still learning the API and such :)
Committed as SVN revision 956927, with the tiny change of "open" rather than "view" in the option label, as that should better indicate that this isn't a read-only operation. Thanks! |