| Summary: | [PATCH] to add wget to the popup menu that shows up when a link is dragged | ||
|---|---|---|---|
| Product: | [Applications] konsole | Reporter: | BOURIAUD <david> |
| Component: | general | Assignee: | Konsole Developer <konsole-devel> |
| Status: | RESOLVED LATER | ||
| Severity: | wishlist | CC: | xavier |
| Priority: | NOR | ||
| Version: | 1.6 | ||
| Target Milestone: | --- | ||
| Platform: | Compiled Sources | ||
| OS: | Linux | ||
| Latest Commit: | Version Fixed In: | ||
| Sentry Crash Report: | |||
*** This bug has been confirmed by popular vote. *** Why not simply use that cp entry in the menu? It uses kfmclient to copy the file to the destination directory... It doesn't support resuming though I think... Because wget can be used on sshed servers too, where kfmclient is often unavailable. Thanks for all the votes and the concern you take on this. In my point of view, the main part is not the use of wget, but the ability to use any program from this context menu. I don't have the skill necessary to write the code needed to have this set up from a config app. Maybe one interested in this could write it for us ? Thanks for your comment anyway. The drag and drop menu is no longer present in KDE 4, so the patch isn't applicable there. This bug does have a fair few votes, but it always struck me that typing "wget" was very easy anyway. I'm closing this bug. I'll commit a fix for 160583 shortly which will bring back the d-n-d menu. I don't see any reason for a special 'wget' entry. You are closing it because you don't want a wget entry? Or did I misunderstood? Regarding the votes, it would be interesting to have what David Bouriaud described in #4. |
Version: 1.6 (using KDE KDE 3.5.0) Installed from: Compiled From Sources Here is a simple diff file to patch TEWidget.cpp so as to add an entry for wget in the popup menu described in subject. -------------------------------------------------------------- 2119c2119 < enum dropPopupOptions { paste, cd, cp, ln, mv }; --- > enum dropPopupOptions { paste, cd, cp, ln, mv, wget }; 2131a2132 > m_drop->insertItem( "wget", wget ); 2150a2152 > m_drop->setItemEnabled( wget, true ); 2227a2230,2232 > case wget: > emit sendStringToEmu("wget "); > break; 2229c2234 < if (item>cd && item<=mv) { --- > if (item>cd && item<=wget) { -------------------------------------------------------------- I've just adjusted the given code to add the functionality I wanted, but it would be great to have this menu be built from a config file. This way, any patch like this dirty one could be avoided, and users could add any app they have/want.