Bug 118857 - [PATCH] to add wget to the popup menu that shows up when a link is dragged
Summary: [PATCH] to add wget to the popup menu that shows up when a link is dragged
Status: RESOLVED LATER
Alias: None
Product: konsole
Classification: Applications
Component: general (show other bugs)
Version: 1.6
Platform: Compiled Sources Linux
: NOR wishlist
Target Milestone: ---
Assignee: Konsole Developer
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2005-12-22 13:58 UTC by BOURIAUD
Modified: 2011-08-14 06:37 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description BOURIAUD 2005-12-22 13:58:36 UTC
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.
Comment 1 Médéric Boquien 2005-12-22 14:45:08 UTC
*** This bug has been confirmed by popular vote. ***
Comment 2 Teemu Rytilahti 2006-01-16 14:29:52 UTC
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...
Comment 3 Amand Tihon 2006-01-16 14:37:40 UTC
Because wget can be used on sshed servers too, where kfmclient is often unavailable.
Comment 4 BOURIAUD 2006-01-26 09:54:02 UTC
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.
Comment 5 Robert Knight 2008-01-12 21:30:34 UTC
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.  
Comment 6 Kurt Hindenburg 2011-08-14 01:05:44 UTC
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.
Comment 7 Xavier Brochard 2011-08-14 06:37:42 UTC
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.