ranting!

[two_pi_r]

So, I use a Soulseek client written in Python. The menu for active downloads partially reads like so:

Abort
Abort and remove files
Retry
Clear

Let's think here. Assuming I have a shitty connection that drops the download every 40k. Which item am I going to pick the most?

Obviously, retry. Which brings up an interesting question!

Why the flying fuck is Retry nestled between Clear, and Abort and fucking Remove The Files I've Been Downloading at 7 Fucking K Per Second?

Granted this is a python application and I could patch it with some d/p action in vim, so I'm just bitching to bitch. Maybe I'll submit the patch and yell at the maintainer.

No sooner said than done.

To: hyriand@thegraveyard.org
Subject: Patch to Nicotine 1.0.8 for Retry menu option
I have a crappy network connection, which occasionally requires that I hit Retry on all my transfers occasionally. In Nicotine 1.0.8, this menu option is right in beteween two undesirable choices - Abort and remove all, and Clear.

So, after almost accidentally clicking "Clear" on all my selected transfers, I finally managed to accidentally click Abort and Remove Files on all my selected transfers. Oops.

So, here is a patch to move the horrendously dangerous options down a few steps and a separator line, making the interface a bit safer.

diff -Naur nicotine-1.0.8_original/pynicotine/gtkgui/downloads.py nicotine-1.0.8/pynicotine/gtkgui/downloads.py
--- nicotine-1.0.8_original/pynicotine/gtkgui/downloads.py 2005-11-14 19:46:19.386420329 -0700
+++ nicotine-1.0.8/pynicotine/gtkgui/downloads.py 2005-11-14 19:47:40.172528810 -0700
@@ -27,11 +27,12 @@
("$" + _("Ban this user"), popup.OnBanUser),
("$" + _("Ignore this user"), popup.OnIgnoreUser),
("", None),
+ (_("Retry"), self.OnRetryTransfer),
+ ("", None),
(_("Abort"), self.OnAbortTransfer),
(_("Abort and remove file(s)"), self.OnAbortRemoveTransfer),
- (_("Retry"), self.OnRetryTransfer),
- (_("Clear"), self.OnClearTransfer),
("", None),
+ (_("Clear"), self.OnClearTransfer),
(_("Clear finished/aborted"), self.OnClearFinishedAborted),
(_("Clear finished"), self.OnClearFinished),
(_("Clear aborted"), self.OnClearAborted),

The patch turns the menu into this: (or fails to apply. My skills with diff run to being able to read the manpage.)

Retry
----------
Abort
Abort and remove files
----------
Clear

(There's 3 more Clear and [do stuff] options at the end of the menu.)

I feel like such a good lunix commie! Aaaaagh, motherland!